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.

Sign up for GitHub

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

Jump to bottom

Request: support .eslint.yaml as a valid config filename #4045

Closed
ronkorving opened this issue Oct 5, 2015 · 4 comments
Closed

Request: support .eslint.yaml as a valid config filename #4045

ronkorving opened this issue Oct 5, 2015 · 4 comments
Assignees
@nzakas
Labels
accepted There is consensus among the team that this change meets the criteria for inclusion archived due to age This issue has been archived; please open a new issue for any further discussion core Relates to ESLint's core APIs and features enhancement This change enhances an existing feature of ESLint
Milestone
v2.0.0

Comments

@ronkorving
Copy link

The reason for this is simple: YAML syntax highlighting in editors.
@eslintbot
Copy link

Thanks for the issue! If you're reporting a bug, please be sure to include:

  1. The version of ESLint you are using (run eslint -v)
  2. What you did (the source code and ESLint configuration)
  3. The actual ESLint output complete with numbers
  4. What you expected to happen instead

Requesting a new rule? Please see Proposing a New Rule for instructions.

@eslintbot eslintbot added the triage An ESLint team member will look at this issue soon label Oct 5, 2015
@ilyavolodin ilyavolodin added enhancement This change enhances an existing feature of ESLint core Relates to ESLint's core APIs and features evaluating The team will evaluate this issue to decide whether it meets the criteria for inclusion and removed triage An ESLint team member will look at this issue soon labels Oct 5, 2015
@nzakas nzakas mentioned this issue Oct 5, 2015
Closed
11 tasks
@nzakas nzakas added accepted There is consensus among the team that this change meets the criteria for inclusion and removed evaluating The team will evaluate this issue to decide whether it meets the criteria for inclusion labels Oct 5, 2015
@nzakas nzakas added this to the v2.0.0 milestone Oct 5, 2015
@nzakas
Copy link
Member

nzakas commented Oct 5, 2015

Yup, this is part of our 2.0 roadmap (#3561), just didn't have an issue for it yet. The intent is to support:

  • .eslintrc.yaml
  • .eslintrc.yml
  • .eslintrc.json

Basically, the first one that is found wins.

@ronkorving
Copy link
Author

Cool, thanks :)

BYK added a commit that referenced this issue Oct 6, 2015
@BYK
Update: support .eslint.yaml as a valid config filename (fixes #4045)
9328bb2
@BYK BYK mentioned this issue Oct 6, 2015
Closed
BYK added a commit that referenced this issue Oct 6, 2015
@BYK
Update: support .eslint.yaml as a valid config filename (fixes #4045)
936b984
BYK added a commit that referenced this issue Oct 10, 2015
@BYK
Update: support .eslint.yaml as a valid config filename (fixes #4045)
d872d9d
@nzakas nzakas self-assigned this Nov 12, 2015
@nzakas
Copy link
Member

nzakas commented Nov 12, 2015

This is turning into quite the refactoring of the config loader. I think that's a good thing. :)

nzakas added a commit that referenced this issue Nov 13, 2015
@nzakas
New: Config files with extensions (fixes #4045)
3504357
nzakas added a commit that referenced this issue Nov 13, 2015
@nzakas
New: Config files with extensions (fixes #4045)
f714ef9
nzakas added a commit that referenced this issue Nov 14, 2015
@nzakas
New: Config files with extensions (fixes #4045, fixes #4263)
0d1c9e1
nzakas added a commit that referenced this issue Nov 14, 2015
@nzakas
New: Config files with extensions (fixes #4045, fixes #4263)
db4bc38
@nzakas nzakas closed this as completed in c9a8883 Nov 16, 2015
nzakas added a commit that referenced this issue Nov 16, 2015
@nzakas
Merge pull request #4406 from eslint/issue4045
756b279 
New: Config files with extensions (fixes #4045)
@eslint-deprecated eslint-deprecated bot locked and limited conversation to collaborators Feb 7, 2018
@eslint-deprecated eslint-deprecated bot added the archived due to age This issue has been archived; please open a new issue for any further discussion label Feb 7, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@nzakas nzakas

Labels
accepted There is consensus among the team that this change meets the criteria for inclusion archived due to age This issue has been archived; please open a new issue for any further discussion core Relates to ESLint's core APIs and features enhancement This change enhances an existing feature of ESLint
Projects
None yet
Milestone
v2.0.0
Development

No branches or pull requests
4 participants
@nzakas @ronkorving @ilyavolodin @eslintbot