-
Notifications
You must be signed in to change notification settings - Fork 699
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
Give up liveness slashing #2254
Conversation
The CI pipeline was cancelled due to failure one of the required jobs. |
I don't get why we need this pr. Why are we not just remove the pallet from the runtime? |
I will adhere to this. Do we really need to change this pallet functionality for a particular use case that can be achieved by just not using it? |
Yeah. I also don't like this and I really don't see any reason. I'm going to close this for the other pr that just removes the stuff. |
In accordance with #1964, we give up liveness slashing. The first step is to always return zero offense threshold. This will be followed up by removing the
im-online
pallet entirely.