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

Add empty_parameters rule #965

Merged
merged 3 commits into from
Dec 14, 2016
Merged

Add empty_parameters rule #965

merged 3 commits into from
Dec 14, 2016

Conversation

marcelofabri
Copy link
Collaborator

Fixes #573

I'm not a big fan of this rule name though 😅

@jpsim
Copy link
Collaborator

jpsim commented Dec 12, 2016

VoidParametersRule?

@marcelofabri
Copy link
Collaborator Author

marcelofabri commented Dec 12, 2016

I thought about that, but should rules describe what they catch or what is the "right way"? 💭

@codecov-io
Copy link

codecov-io commented Dec 12, 2016

Current coverage is 81.67% (diff: 92.85%)

Merging #965 into master will decrease coverage by 0.05%

@@             master       #965   diff @@
==========================================
  Files           132        134     +2   
  Lines          6395       6282   -113   
  Methods           0          0          
  Messages          0          0          
  Branches          0          0          
==========================================
- Hits           5227       5131    -96   
+ Misses         1168       1151    -17   
  Partials          0          0          

Powered by Codecov. Last update 8b49cd3...dc50204

@marcelofabri marcelofabri mentioned this pull request Dec 12, 2016
@jpsim
Copy link
Collaborator

jpsim commented Dec 12, 2016

I actually like the "Empty Parameters" rule name.

@marcelofabri
Copy link
Collaborator Author

Let's keep then! I'll rebase the changelog

@jpsim
Copy link
Collaborator

jpsim commented Dec 12, 2016

I'll mark this as [WIP] until you can adapt to ignore "middle types" as discussed in #967 (comment)

@jpsim jpsim changed the title Add empty_parameters rule [WIP] Add empty_parameters rule Dec 12, 2016
@marcelofabri marcelofabri changed the title [WIP] Add empty_parameters rule Add empty_parameters rule Dec 12, 2016
@jpsim
Copy link
Collaborator

jpsim commented Dec 14, 2016

🎉

@jpsim jpsim merged commit 6665986 into realm:master Dec 14, 2016
@marcelofabri marcelofabri deleted the empty-parameters-rule branch December 14, 2016 23:39
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