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

[Refactor] Use ESLint instead of StandarJS and add settings for TravisCI #231

Merged
merged 13 commits into from
Jan 11, 2017

Conversation

sota1235
Copy link
Contributor

Update

  • Use ESLint for checking coding style.
  • Add config for using TravisCI.
  • Ignore invalid tests. Fix them after...

@sota1235 sota1235 self-assigned this Jan 11, 2017
@sota1235 sota1235 merged commit f70cf78 into BoostIO:master Jan 11, 2017
@asmsuechan
Copy link
Contributor

Hi @sota1235, after this PR was merged, a hot key Toggle Finder(popup) doesn't work properly at least in my environment. Can you check it?

@asmsuechan
Copy link
Contributor

This commit 5c7804f is doubtful.

@sota1235
Copy link
Contributor Author

Oh, I'm sorry. I'll fix it 🙇

@sota1235
Copy link
Contributor Author

I revert this commit, but I can't fix this bug.
I try to investigate this bug and fix it on next release.

@asmsuechan
Copy link
Contributor

asmsuechan commented Jan 14, 2017

In my environment, I could fix it by revert the commit. Did you try to restart electron? However the lint fails.

@asmsuechan
Copy link
Contributor

Oh, I also fail to fix it by revert now 😨

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