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

Update geonodes.toml #3300

Closed
wants to merge 1 commit into from
Closed

Update geonodes.toml #3300

wants to merge 1 commit into from

Conversation

gbuchukuri
Copy link
Contributor

@gbuchukuri gbuchukuri commented Dec 14, 2023

Description

Update geonodes.toml for testnet-15

Links to previous PRs

#237
#1059
#1474
#1800

Rules

All previous genesis validators should name their PRs "Update {validator_alias}.toml" and provide links to previous PRs merged.

All new validators should name their PRs "Create {validator_alias}.toml" and provide links to their previously closed PRs, if any.

Checklist before merging

  • Only one toml is added in this PR
  • The file being added has no special characters in its name (no '.' or ' ')
  • The file being added is indeed a .toml file
  • The toml being added is to the correct folder, and only to the correct folder
  • The eth_hot_key and eth_cold_key are present
  • The email is present

@bengtlofgren
Copy link
Collaborator

You messed up the filename buddy

@bengtlofgren
Copy link
Collaborator

Closed in favour of #3317
But this is going to be the last time I ask you to get the filename correct. If you don't know what I'm talking about, ask people in the discord but your filename is "namada-public-testnet-15/Update Geonodes.toml". That is not correct. See https://github.com/anoma/namada-testnets/pull/3300/files (INCORRECT)
vs
https://github.com/anoma/namada-testnets/pull/3317/files (CORRECT)

@gbuchukuri
Copy link
Contributor Author

gbuchukuri commented Dec 15, 2023 via email

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.

2 participants