Skip to content
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

✨ Add Json Schema Generation #64

Merged
merged 10 commits into from
Aug 9, 2023
Merged

Conversation

NotJeremyLiu
Copy link
Member

@NotJeremyLiu NotJeremyLiu commented Aug 9, 2023

This PR:

  • Adds schema gen files to all contracts
  • Adds a make command to generate entry point schema (only entry point is necessary since that is the only contract callable by external parties)
  • Adds a CI check to ensure the schema is up to date given any changes

NOTE: Tried to use https://github.com/omissis/go-jsonschema to generate go types from the json schema, but it only generates the types themselves and not the Msgs. For example, there is no ExecuteMsg enum generated into go code. So for now having the json schemas will have to do.

@NotJeremyLiu NotJeremyLiu changed the title Jl/add json schema generation ✨ Add Json Schema Generation Aug 9, 2023
@NotJeremyLiu NotJeremyLiu merged commit 1a57cec into main Aug 9, 2023
4 checks passed
@NotJeremyLiu NotJeremyLiu deleted the jl/add-json-schema-generation branch August 9, 2023 19:18
@NotJeremyLiu NotJeremyLiu mentioned this pull request Sep 12, 2023
15 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant