New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Rename #type
to #schema
and introduce #schema_id
#3183
Conversation
d1154b2
to
da6467d
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That was fast. I left two comments for your two TODOs, which I think should be addressed. I'd then like to take another look, but this should be good to go otherwise.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Perfect, thanks! Let's get this merged quickly so you don't have to resolve many merge conflicts for this.
40b57be
to
8eaa392
Compare
We introduced a breaking change that is not backwards-compatible, so from now on the regression tests will only cover v4.0.0 and upwards, but also include the version from the PR itself.
I'm merging this with a manual CI override because the regression test for latest cannot succeed before this is actually merged and the image was uploaded once on main. |
No description provided.