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

eslint config gets linked or copied depending on where it comes from #40

Merged
merged 5 commits into from
Aug 20, 2016

Conversation

ccpricenytimes
Copy link
Contributor

@ccpricenytimes ccpricenytimes commented Aug 16, 2016

Adds an eslint.json to user's repo:
copies from a starter-kyt if one exists (this is something that would go hand in hand with a starter-kyts kyt.config.js)
or
copies default from kyt

connects #47

} else {
// Create a symbolic link from our local eslint
const esLintPath = path.join(userRootPath, 'node_modules/kyt/eslint.json');
if (shell.ln('-s', esLintPath, linkedPath).code === 0) {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Symlink needs to be relative, see #44

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

will do thanks!

@ccpricenytimes
Copy link
Contributor Author

ok Adding a few updates here

  1. the ESLint file now gets copied instead of symlinked so users can just edit rather than merge.
  2. kyt's default ESLint file now includes rules from wf-style cc. @tizmagik

one note:
the linter is printing out a warning due to an outdated rule in airbnb. See airbnb/javascript#978

@ccpricenytimes
Copy link
Contributor Author

@tizmagik I can't figure out to reply to our original thread. I'm going work on making setup interactive as a whole so I'd like to defer your note until then. that ok?

@tizmagik
Copy link
Contributor

@ccpricenytimes oh that's great! Yea that sounds good!

@delambo delambo merged commit ffaf3d9 into master Aug 20, 2016
@delambo delambo deleted the esLintLink branch August 20, 2016 16: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.

3 participants