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
Remove JSON schemas in favour of structs #862
Conversation
0d0c74c
to
132fbdb
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.
I'm generally all for this change.
Could we talk through implementing / not implementing https://github.com/MetaMask/SIPs/blob/main/SIPS/sip-4.md before merging this though? No need to duplicate efforts
The work for this PR is already done. If we decide on merging the two, we'd need to redo the work regardless, so I don't think it makes any difference. |
5910849
to
c7e974d
Compare
This replaces the
package.json
andsnap.manifest.json
JSON schemas in favour ofsuperstruct
.To do