-
Notifications
You must be signed in to change notification settings - Fork 570
refactor(network): upgrade spn
#2816
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
Conversation
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.
lgtm
|
Changelog? |
I will continue to work on this PR for some of the other changes so it can remain open for review without me being blocked. Btw I don't think we need to enforce changelog for each PR, sometimes it's not relevant and the changelog file shouldn't be too much extensive |
…nto network/launch-time
network): upgrade spn and replace remaining-time by launch-timenetwork): upgrade spn
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.
LGTM 👍
Co-authored-by: Thomas Bruyelle <thomas.bruyelle@gmail.com>
* update go mod * clock package * refactor network * new cmd * last changes * typoe * import * fix unit test * fix test 2 * Upgrade for send request * publish: add balance * new logic for adding accounts * format * change switch * changelog * Update ignite/cmd/network_chain_join.go Co-authored-by: Thomas Bruyelle <thomas.bruyelle@gmail.com> Co-authored-by: Alex Johnson <alex@shmeeload.xyz> Co-authored-by: Thomas Bruyelle <thomas.bruyelle@gmail.com>
Part of #2812
remaining-timeis replaced by absolutelaunch-timeMsgSendRequest--account-balanceflag inpublishfor the coordinator to fix genesis account balancejoinif it existsTesting
SPN:
Ignite