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

docs: update changelog #2694

Closed
wants to merge 2 commits into from
Closed

docs: update changelog #2694

wants to merge 2 commits into from

Conversation

fadeev
Copy link
Contributor

@fadeev fadeev commented Aug 4, 2022

Fixes

  • Make go mod verify optional, so Ignite CLI can work in a Go workspace context #2685
  • The monitoringp module has been removed from the default template (removed dependency on tenderming/spn) #2648
  • Added a call to RegisterMsgServer in the default module's template (to match what's specified in the docs) #2636
  • Fix query scaffolding by using the correct query context in the chain's CLI package #2668
  • Updated keyring dependency to match what Cosmos SDK is using #2655

@fadeev
Copy link
Contributor Author

fadeev commented Aug 18, 2022

Closing, because of #2757

@ilgooz ilgooz deleted the docs-changelog branch August 18, 2022 14:23
@ilgooz
Copy link
Member

ilgooz commented Aug 18, 2022

Should we add the ones in here to that PR?

@ilgooz
Copy link
Member

ilgooz commented Aug 18, 2022

cc @jeronimoalbi

@jeronimoalbi
Copy link
Member

Should we add the ones in here to that PR?

I will add the missing ones to the other PR.

By the way, we are currently not using the hash plus issue number in the changelog. We could start adding them, at least the #NUMBER, or is there a reason why we are not?

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.

3 participants