Skip to content

Make artifacts.*.type optional in bundle JSON schema.#2881

Merged
shreyas-goenka merged 3 commits intomainfrom
opt-artifact-path
May 14, 2025
Merged

Make artifacts.*.type optional in bundle JSON schema.#2881
shreyas-goenka merged 3 commits intomainfrom
opt-artifact-path

Conversation

@shreyas-goenka
Copy link
Copy Markdown
Contributor

@shreyas-goenka shreyas-goenka commented May 14, 2025

Why

It's not required for artifacts defined in DABs. We only support the "whl" value for the type field in artifacts.

Tests

Generated schema has type no longer required.

@shreyas-goenka shreyas-goenka marked this pull request as ready for review May 14, 2025 12:14
@shreyas-goenka shreyas-goenka enabled auto-merge May 14, 2025 12:29
@shreyas-goenka shreyas-goenka added this pull request to the merge queue May 14, 2025
Merged via the queue into main with commit b60a69f May 14, 2025
10 checks passed
@shreyas-goenka shreyas-goenka deleted the opt-artifact-path branch May 14, 2025 13:11
deco-sdk-tagging bot added a commit that referenced this pull request May 14, 2025
## Release v0.252.0

### Dependency updates
* Upgraded Go SDK to 0.69.0 ([#2867](#2867))
* Upgraded to TF provider 1.79.0 ([#2869](#2869))

### Bundles
* Remove unused fields from resources.models schema: creation\_timestamp, last\_updated\_timestamp, latest\_versions and user\_id. Using them now raises a warning ([#2828](#2828)).
* Preserve folder structure for app source code in bundle generate ([#2848](#2848))
* Fix normalising requirements file path in dependencies section ([#2861](#2861))
* Fix default-python template not to add environments when serverless=yes and include\_python=no ([#2866](#2866))
* Fix handling of Unicode characters in Python support ([#2873](#2873))
* Add support for secret scopes in DABs ([#2744](#2744))
* Make `artifacts.*.type` optional in bundle JSON schema ([#2881](#2881))
* Fix support for `spot_bid_max_price` field in Python support ([#2883](#2883))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants