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

Better Linting and Some 💄 #113

Merged
merged 1 commit into from
Mar 20, 2017
Merged

Better Linting and Some 💄 #113

merged 1 commit into from
Mar 20, 2017

Conversation

evilsoft
Copy link
Owner

Spring has sprung, time to 💅

image

Thought there were default options on and was wondering why some of my code was passing lint when it was horrible. Turns out, things MUST be specified. This PR update the lint options and adjusts the existing code base accordingly,

@coveralls
Copy link

coveralls commented Mar 20, 2017

Coverage Status

Coverage remained the same at 100.0% when pulling 3c9deda on lint-unused into 73a0f4f on master.

@evilsoft
Copy link
Owner Author

image

@evilsoft evilsoft merged commit 0cd2770 into master Mar 20, 2017
@evilsoft evilsoft deleted the lint-unused branch March 20, 2017 02:36
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.

2 participants