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

removed unnecessary inheritance of M2 filtering arguments collection #5498

Merged
merged 1 commit into from
Dec 8, 2018

Conversation

davidbenjamin
Copy link
Contributor

@takutosato this will remove some confusing irrelevant arguments.

Copy link
Contributor

@takutosato takutosato left a comment

Choose a reason for hiding this comment

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

This will make @meganshand happy

@davidbenjamin
Copy link
Contributor Author

In ten days or so there will be a lot more along these lines.

@codecov-io
Copy link

Codecov Report

Merging #5498 into master will decrease coverage by 0.001%.
The diff coverage is 100%.

@@               Coverage Diff               @@
##              master     #5498       +/-   ##
===============================================
- Coverage     87.063%   87.062%   -0.001%     
  Complexity     31256     31256               
===============================================
  Files           1922      1922               
  Lines         144305    144305               
  Branches       15918     15918               
===============================================
- Hits          125636    125635        -1     
- Misses         12891     12892        +1     
  Partials        5778      5778
Impacted Files Coverage Δ Complexity Δ
...ls/walkers/mutect/M2FiltersArgumentCollection.java 100% <100%> (ø) 1 <1> (ø) ⬇️
...oadinstitute/hellbender/utils/gcs/BucketUtils.java 79.878% <0%> (-0.61%) 42% <0%> (ø)

@davidbenjamin davidbenjamin merged commit ae33cd1 into master Dec 8, 2018
@davidbenjamin davidbenjamin deleted the db_m2_filter_args branch December 8, 2018 22:47
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.

3 participants