-
Notifications
You must be signed in to change notification settings - Fork 176
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
Update cw-plus packages to 0.15.0 #80
Conversation
Oh great, cw-nfts may be one of the first one to update. Cutting edge. 😄 Looking forward to this. (It provides much better schemas and will make ts-codegen happy) |
Going to need a rebase. 😂 |
You just did a big merge? |
c37720a
to
dd5e0cc
Compare
52de28d
to
a1325ee
Compare
509db67
to
389ca62
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.
What I see looks good.
What I don't see are the changes to schema.rs
and moving them to the bin directory and such. I guess the movement is optional, but I think a big part of the upgrade is also using these unified schema output (the new format from Tom).
I am happy to approve and merge this as is. But I would wait until those schema.rs files are modified to tag the repo and make a v0.15 release
@@ -76,7 +76,8 @@ | |||
], | |||
"properties": { | |||
"never": { | |||
"type": "object" | |||
"type": "object", | |||
"additionalProperties": false |
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.
nice!
5eec9bd
to
ccb105d
Compare
No description provided.