Skip to content

feat!: convert parameter schema from voluptuous to msgspec#940

Merged
abhishekmadan30 merged 2 commits intotaskcluster:mainfrom
abhishekmadan30:parameter-schema
May 1, 2026
Merged

feat!: convert parameter schema from voluptuous to msgspec#940
abhishekmadan30 merged 2 commits intotaskcluster:mainfrom
abhishekmadan30:parameter-schema

Conversation

@abhishekmadan30
Copy link
Copy Markdown
Contributor

No description provided.

@abhishekmadan30 abhishekmadan30 requested a review from a team as a code owner April 29, 2026 17:50
@abhishekmadan30 abhishekmadan30 requested a review from hneiva April 29, 2026 17:50
@abhishekmadan30 abhishekmadan30 added the BREAKING CHANGE Backwards incompatible request that will require major version bump label Apr 29, 2026
@abhishekmadan30 abhishekmadan30 force-pushed the parameter-schema branch 2 times, most recently from a498061 to 07a766f Compare April 29, 2026 19:51
Copy link
Copy Markdown
Collaborator

@ahal ahal left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, looks good!

I'd like to standardize on the from_dict format though, so maybe we can land new code in that format.

Comment thread src/taskgraph/parameters.py Outdated
Comment thread taskcluster/self_taskgraph/custom_parameters.py Outdated
Copy link
Copy Markdown
Collaborator

@ahal ahal left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, lgtm!

@ahal
Copy link
Copy Markdown
Collaborator

ahal commented Apr 30, 2026

One minor nit, but if we're following conventional commits, the prefix should be feat! (exclamation mark means breaking). Also there's supposed to be somehting like:

BREAKING CHANGE: <reason>

in the commit body. No big deal because we don't enforce it here, so more of an fyi

@abhishekmadan30 abhishekmadan30 changed the title feat: convert parameter schema from voluptuous to msgspec feat!: convert parameter schema from voluptuous to msgspec May 1, 2026
@abhishekmadan30 abhishekmadan30 force-pushed the parameter-schema branch 2 times, most recently from ac51cf1 to b3e0a05 Compare May 1, 2026 15:43
@abhishekmadan30 abhishekmadan30 merged commit b4aa5f5 into taskcluster:main May 1, 2026
20 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

BREAKING CHANGE Backwards incompatible request that will require major version bump

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants