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

style(all): beautify code #122

Merged
merged 2 commits into from
Jan 13, 2017
Merged

style(all): beautify code #122

merged 2 commits into from
Jan 13, 2017

Conversation

pidupuis
Copy link
Collaborator

@pidupuis pidupuis commented Jan 12, 2017

Add linter utility, fix indent, fix mixed spaces, use single quotes, use strict equality, fix broken tags, ...

@Halyul
Copy link
Contributor

Halyul commented Jan 13, 2017

Does this fix issue #70 ?

@pidupuis
Copy link
Collaborator Author

I don't think so.

This PR is only focused on improving the code quality. There is no new features, nor fixes. I requested reviews to check for code regression but in theory there is no differences in term of functionalities.

@iblh iblh changed the title Beautify code style(all): beautify code Jan 13, 2017
@iblh iblh merged commit f60bdb1 into canary Jan 13, 2017
@iblh iblh mentioned this pull request Jan 13, 2017
@iblh iblh deleted the refactor/lint branch January 13, 2017 12:31
@iblh iblh restored the refactor/lint branch January 13, 2017 12:31
@pidupuis pidupuis deleted the refactor/lint branch January 13, 2017 15:26
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.

None yet

4 participants