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

🎨 Clean up and refactor to remove @codingStandardsIgnore annotation #52

Merged
merged 4 commits into from
Dec 29, 2016
Merged

🎨 Clean up and refactor to remove @codingStandardsIgnore annotation #52

merged 4 commits into from
Dec 29, 2016

Conversation

winkbrace
Copy link

I noticed you had to ignore a part of the IndentationLevelSniff, so I took it upon me to refactor it a bit to get rid of the annotation.

I hope you're not unaccustomed with some functional programming 😅

@codecov-io
Copy link

codecov-io commented Dec 29, 2016

Current coverage is 99.24% (diff: 100%)

Merging #52 into master will increase coverage by 0.01%

@@             master        #52   diff @@
==========================================
  Files             8          8          
  Lines           130        133     +3   
  Methods          25         25          
  Messages          0          0          
  Branches          0          0          
==========================================
+ Hits            129        132     +3   
  Misses            1          1          
  Partials          0          0          

Powered by Codecov. Last update 53fb917...d470cb7

@bmitch bmitch merged commit 4f1650f into bmitch:master Dec 29, 2016
@bmitch
Copy link
Owner

bmitch commented Dec 29, 2016

Thanks @winkbrace !

@bmitch
Copy link
Owner

bmitch commented Dec 29, 2016

@winkbrace if you are up for it I wonder if the same technique could be used here?

https://github.com/bmitch/Codor/blob/master/src/Codor/Sniffs/Files/ReturnNullSniff.php#L22

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