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

Remove unused slashing protos #5308

Merged
merged 2 commits into from
Apr 5, 2020

Conversation

0xKiwi
Copy link
Contributor

@0xKiwi 0xKiwi commented Apr 4, 2020

Removes unused protos from the slasher, such as min max span protos and slashing response.

@codecov
Copy link

codecov bot commented Apr 4, 2020

Codecov Report

Merging #5308 into master will not change coverage by %.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #5308   +/-   ##
=======================================
  Coverage   36.53%   36.53%           
=======================================
  Files         239      239           
  Lines       22485    22485           
=======================================
  Hits         8216     8216           
  Misses      12825    12825           
  Partials     1444     1444           

Copy link
Contributor

@shayzluf shayzluf left a comment

Choose a reason for hiding this comment

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

lgtm

@0xKiwi 0xKiwi merged commit 0775318 into prysmaticlabs:master Apr 5, 2020
@0xKiwi 0xKiwi mentioned this pull request Apr 14, 2020
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