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 ignore-prefix option to no-object-mutation rule #53

Merged
merged 1 commit into from
Sep 27, 2017

Conversation

miangraham
Copy link
Contributor

Added an option for ignoring object mutation by prefix. Partially addresses #43.

Feels like the option parsing and isIgnored checks want a shared module like readonly-shared to avoid duplication, but made them their own thing for now to avoid affecting other rules.

@coveralls
Copy link

coveralls commented Sep 27, 2017

Coverage Status

Coverage decreased (-0.9%) to 96.909% when pulling 13b159e on miangraham:master into eb57489 on jonaskello:master.

@jonaskello
Copy link
Owner

I think the branch where there are multiple ignore-prefix is not covered by the tests, that's why coveralls complains. However this should not be a problem so I will go ahead and merge anyway :-).

@jonaskello jonaskello merged commit ca2bb32 into jonaskello:master Sep 27, 2017
@jonaskello
Copy link
Owner

jonaskello commented Sep 27, 2017

And yes, you are right about the readonly-shared module. I will go ahead and refactor that for better re-usability when I get some spare time.

Thanks for the contribution :-)

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