feat!: add support for msgspec schemas#844
Conversation
b836118 to
e9b2333
Compare
|
I see this is failing the free-threading tests; I think that's probably fine - it's not officially supported here yet. AFAICT, free-threading is supported in the unreleased version of msgspec, and they expect to release soon, so we can pick it up when they do. |
Over in taskcluster#844, free-threaded python tests are causing `complete-pr` not to run because `msgspec` hasn't released a version that supports it yet. This highlights the fact that free-threading is not officially supported in Taskgraph yet, so we shouldn't block other tasks on these tests.
Over in taskcluster#844, free-threaded python tests are causing `complete-pr` not to run because `msgspec` hasn't released a version that supports it yet. This highlights the fact that free-threading is not officially supported in Taskgraph yet, so we shouldn't block other tasks on these tests.
Over in #844, free-threaded python tests are causing `complete-pr` not to run because `msgspec` hasn't released a version that supports it yet. This highlights the fact that free-threading is not officially supported in Taskgraph yet, so we shouldn't block other tasks on these tests.
ahal
left a comment
There was a problem hiding this comment.
Per https://www.conventionalcommits.org/en/v1.0.0/, please make the prefix feat!: to denote that it's a breaking change. I'd also make the summary shorter, and include the extra context about LegacySchema in a new paragraph.
If you could also update docs/reference/migrations.rst with how to deal with the backwards incompatibility, that would be appreciated! Just make a new 18.x section
Over in taskcluster#844, free-threaded python tests are causing `complete-pr` not to run because `msgspec` hasn't released a version that supports it yet. This highlights the fact that free-threading is not officially supported in Taskgraph yet, so we shouldn't block other tasks on these tests.
|
s/volutpous/voluptuous/ please :) |
|
msgspec support for 3.14 has been released https://github.com/jcrist/msgspec/releases/tag/0.20.0 |
Over in taskcluster#844, free-threaded python tests are causing `complete-pr` not to run because `msgspec` hasn't released a version that supports it yet. This highlights the fact that free-threading is not officially supported in Taskgraph yet, so we shouldn't block other tasks on these tests.
4e9cbed to
6659cc7
Compare
40ece2e to
942de4e
Compare
ahal
left a comment
There was a problem hiding this comment.
I'll stamp one more time. Just double check there weren't any new Schema instances that got added since the last time.
Also a nitpick, the commit summary (first line) is a bit verbose. Consider feat!: add support for msgspec schemas
Then bot the refactoring info in the description part of the commit message.
942de4e to
dab3de9
Compare
dab3de9 to
630734f
Compare
No description provided.