-
Notifications
You must be signed in to change notification settings - Fork 267
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 Upnode gentx #440
Add Upnode gentx #440
Conversation
please make sure your gentx is formatted. These lines are failing:
there shouldn't be empty lines. |
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.
signature verification failed; please verify account number (0), sequence (0) and chain-id (celestia): unauthorized
please make sure the correct chain-id was used
removing the additional block, but we still need to fix conflicts and CI before we merge
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.
- sync your fork: https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/working-with-forks/syncing-a-fork
- merge master into your branch
patch-1
- move your peer into a separate file: under
celestia
, e.g.celestia/upnode.txt
(as described here as well) - make sure to not edit
peers.txt
(aka there should be no diff between peers.txt on master and in your branch)
Thank you
Head branch was pushed to by a user without write access
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.
🎉
Congrats, your important contribution to this open-source project has earned you a GitPOAP! GitPOAP: 2023 Celestia Contributor: Head to gitpoap.io & connect your GitHub account to mint! Learn more about GitPOAPs here. |
Overview
Add Upnode gentx and peer
Checklist