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

Bengt/Add unjail validator CLI command #1654

Merged
merged 8 commits into from
Jul 22, 2023
Merged

Conversation

bengtlofgren
Copy link
Contributor

@bengtlofgren bengtlofgren commented Jul 4, 2023

We only had an SDK implementation by accident

adrianbrink
adrianbrink previously approved these changes Jul 4, 2023
Cargo.toml Outdated
@@ -130,7 +130,7 @@ tonic = "0.8.3"
tonic-build = "0.8.4"
tower = "0.4"
# Also, using the same version of tendermint-rs as we do here.
tower-abci = {git = "https://github.com/heliaxdev/tower-abci.git", rev = "367d8d958b83c501ed2c09e9c4595f8bf75a0b01"}
tower-abci = {git = "https://github.com/heliaxdev/tower-abci.git", rev = "3acd063da2f251d7f162c141a3e77d3271c655a6"}
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this was temp and should be undone

@brentstone brentstone force-pushed the bengt/add-unjail-validator branch from 4bf670a to a5d2237 Compare July 11, 2023 18:14
@brentstone brentstone requested a review from tzemanovic July 11, 2023 18:18
brentstone added a commit that referenced this pull request Jul 12, 2023
* bengt/add-unjail-validator:
  handle errors for unjail-validator tx in the client
  expand and fix e2e test `double_signing_gets_slashed`
  add unjail tx at CLI
  changelog: add #1656
  pos: return sorted validator sets and code re-use for queries
  Expanding and fixing slashes query
  CLI query a validator's state
@brentstone brentstone mentioned this pull request Jul 12, 2023
@Fraccaman Fraccaman merged commit 7a18295 into main Jul 22, 2023
@Fraccaman Fraccaman deleted the bengt/add-unjail-validator branch July 22, 2023 11:41
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.

5 participants