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

.eslintrc should have extension #62

Closed
GitTom opened this issue May 10, 2016 · 3 comments
Closed

.eslintrc should have extension #62

GitTom opened this issue May 10, 2016 · 3 comments

Comments

@GitTom
Copy link

GitTom commented May 10, 2016

This is nit-picky, but the use of an extensionless eslint is deprecated, and it should be super simple to change.

v1.10.0 introduces the ability to use configuration files in different formats. Instead of the regular .eslintrc file, you can use a JavaScript (.eslintrc.js), a YAML file (.eslintrc.yml or .eslintrc.yaml), or a JSON file (.eslintrc.json). We are formally deprecating use of the .eslintrc extensionless configuration file format in favor the format-specific versions. Don't worry, we'll still support .eslintrc files for a long time, but we'd like to encourage everyone to move to the new file formats as you'll get advantages such as syntax highlighting and error detection with many editors.

(bold added by me)

@joshskeen
Copy link
Collaborator

Nice, thanks for the heads up on this! Feel free to submit a PR fixing.

@GitTom
Copy link
Author

GitTom commented May 12, 2016

I'm not actually using alexa-app (yet) and am rather a git newbie so I'll leave that to others.

@joshskeen
Copy link
Collaborator

matt-kruse@b2a2571

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

No branches or pull requests

2 participants