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

Rewrite ExplicitInitRule as ASTRule #826

Merged
merged 4 commits into from
Oct 29, 2016

Conversation

norio-nomura
Copy link
Collaborator

@norio-nomura norio-nomura commented Oct 4, 2016

Rewrite #715 as ASTRule for reducing false positive.
/cc: @mtaube, @jpsim

@codecov-io
Copy link

codecov-io commented Oct 4, 2016

Current coverage is 84.71% (diff: 92.98%)

Merging #826 into master will increase coverage by 0.10%

@@             master       #826   diff @@
==========================================
  Files           105        106     +1   
  Lines          4332       4389    +57   
  Methods           0          0          
  Messages          0          0          
  Branches          0          0          
==========================================
+ Hits           3665       3718    +53   
- Misses          667        671     +4   
  Partials          0          0          

Powered by Codecov. Last update 21e3439...9b64c7d

@norio-nomura norio-nomura added the enhancement Ideas for improvements of existing features and rules. label Oct 8, 2016
@norio-nomura norio-nomura force-pushed the rewrite-explicit-init-rule-to-astrule branch from d3e556d to 9b64c7d Compare October 29, 2016 05:33
@norio-nomura norio-nomura changed the title Rewrite ExplicitInitRule to ASTRule Rewrite ExplicitInitRule as ASTRule Oct 29, 2016
@norio-nomura norio-nomura merged commit 106be03 into master Oct 29, 2016
@norio-nomura norio-nomura deleted the rewrite-explicit-init-rule-to-astrule branch October 29, 2016 05:47
This was referenced Oct 29, 2016
@jpsim
Copy link
Collaborator

jpsim commented Nov 25, 2016

Seems ok. Thanks, @norio-nomura and @mtaube

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Ideas for improvements of existing features and rules.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants