-
Notifications
You must be signed in to change notification settings - Fork 3.8k
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
chore: remove crisis
from simapp
and deprecate
#23722
base: release/v0.53.x
Are you sure you want to change the base?
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.
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.
wait can you mark deprecated or remove here
Line 12 in 6403f7c
* [Crisis](./crisis/README.md) - Halting the blockchain under certain circumstances (e.g. if an invariant is broken). |
some crisis data here:
|
couple of occurrences of crisis here cosmos-sdk/x/upgrade/README.md Line 249 in e5781a9
|
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.
we should make a note in the x/crisis README.md as deprecated
theres still some "crisis" in x/upgrade/readme.md, here, here, here
may want to remove this
|
makes perfect sense. Doesn't work and potentially dangerous if used. |
simapp