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
Remove stray .eslintrc.yml file #6685
Conversation
@irvinebroque, thanks for your PR! By analyzing the annotation information on this pull request, we identified @vitorbal to be a potential reviewer |
Thanks for the pull request, @irvinebroque! I took a look to make sure it's ready for merging and found some changes are needed:
Can you please update the pull request to address these? (More information can be found in our pull request guide.) |
Thank you for your pull request. It looks like this may be your first contribution to a jQuery Foundation project, if so we need you to sign our Contributor License Agreement (CLA). 📝 Please visit http://contribute.jquery.org/CLA/ to sign. After you signed, the PR is checked again automatically after a minute. If there's still an issue, please reply here to let us know. If you've already signed our CLA, it's possible your git author information doesn't match your CLA signature (both your name and email have to match), for more information, check the status of your CLA check. |
Thanks for the pull request, @irvinebroque! I took a look to make sure it's ready for merging and found some changes are needed:
Can you please update the pull request to address these? (More information can be found in our pull request guide.) |
LGTM |
Looks like this is due to an internal rule we introduced for our own rule validation. My guess is we have logic which iterates over all files in lib/rules when @vitorbal Thoughts? |
Ack, meant to comment on the issue. For the PR, I would say this is not the right fix because the YAML is intentional, not stray. |
Agree. This is not the right way to fix this. We should move that .eslintrc.yml file out rules directory, or change our code that works for picking up all of the rules. We will also have to change some other parts of code then (like deployment scripts). |
Let's just keep it simple: ensure the code that creates |
@nzakas We also iterate through all of the files in rules directory as part of the release script here: https://github.com/eslint/eslint/blob/master/Makefile.js#L139 It doesn't seem to be causing issue with the demo, but we should probably modify it as well. |
I am also getting this issue |
Oops, I'm very sorry for this regression :( @ilyavolodin It seems like all the places in the Makefile that iterate through the files in the rules directory also filter out non ".js" files before doing anything, including the spot you mentioned in #6685 (comment). Is there any spots I might have missed? |
@vitorbal Probably not, I just remembered that we are iterating through that directory in more then one place. |
This is also the cause of sarbbottam/eslint-find-rules#132, fwiw |
Since eslint doesn't export a list of rules for people to rely on, it's likely that many tools will resort to this mechanism for locating them. Perhaps a better long term solution is to export a list of all rules, so that others can rely on that instead? |
Closing in favor of #6696 |
What issue does this pull request address?
A regression in v3.1.0 causing the following eslint error to be thrown in every file:
2:1 error Definition for rule '.eslintrc.yml' was not found .eslintrc.yml
Given the following config
What changes did you make? (Give an overview)
I removed what appears to be a stray
.eslintrc.yml
file in thelib/rules
directory.Is there anything you'd like reviewers to focus on?
Is this
.eslintrc.yml
file actually needed?