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

Added extension to .eslintrc #5988

Merged
merged 1 commit into from Dec 7, 2018
Merged

Conversation

@ManoelLobo
Copy link
Contributor

@ManoelLobo ManoelLobo commented Dec 7, 2018

.eslintrc files without extension are deprecated

And thanks for this amazing tool!

@Timer Timer added the tag: internal label Dec 7, 2018
@Timer Timer added this to the 2.1.2 milestone Dec 7, 2018
@Timer
Copy link
Collaborator

@Timer Timer commented Dec 7, 2018

Neat, thanks! I'll wait for CI to pass.

Copy link
Collaborator

@mrmckeb mrmckeb left a comment

Good pickup, thanks @ManoelLobo. I'll give this a quick test today or tomorrow and then merge if everything is OK.

Could you also update this page?
https://facebook.github.io/create-react-app/docs/setting-up-your-editor#displaying-lint-output-in-the-editor

Edit: I'll leave this with @Timer - sorry for the double review 😉 - but it would be great to update the docs too.

@ManoelLobo
Copy link
Contributor Author

@ManoelLobo ManoelLobo commented Dec 7, 2018

Sure, @mrmckeb, was already looking into it 😃 . As it is documentation-related, I think it is better to open another PR. Or should add more commits to this one?

@mrmckeb
Copy link
Collaborator

@mrmckeb mrmckeb commented Dec 7, 2018

I'd say one PR, as the doc update is fairly minor and related, but @Timer might have a different opinion?

@Timer
Copy link
Collaborator

@Timer Timer commented Dec 7, 2018

We can run with a separate PR -- this one is just internal for the monorepo. :-)

Feel free to just use the GH edit button to fix the typo in the docs.

Also, any app created with v2 automatically configures your Editor's linting so following that guide is unnecessary. Maybe we should note that. That PR might have more back and forth before it's finalized.

Thanks!

@Timer Timer merged commit fbdf156 into facebook:master Dec 7, 2018
1 of 3 checks passed
1 of 3 checks passed
continuous-integration/appveyor/pr Waiting for AppVeyor build to complete
Details
continuous-integration/travis-ci/pr The Travis CI build is in progress
Details
deploy/netlify Docs deploy preview succeeded
Details
@ManoelLobo ManoelLobo deleted the ManoelLobo:pr/eslintrc-extension branch Dec 7, 2018
timsnadden added a commit to timsnadden/create-react-app that referenced this pull request Dec 7, 2018
* master: (39 commits)
  Added extension to .eslintrc (facebook#5988)
  Update links to docs in all package README files (facebook#5912)
  Use https for linked images in docs to fix mixed content warnings (facebook#5985)
  Improve error messaging in verifyPackageTree.js (facebook#5974)
  Add removeItem to localStorage mock in docs (facebook#5919)
  Add SASS_PATH instructions to Sass docs (facebook#5917)
  Suggest a different default for speed reasons (facebook#5959)
  Add pre-eject message about new features in v2 (facebook#5954)
  Add netlify.toml to prepare for deploy to netlify facebook#5807 (facebook#5930)
  Correct some comments (facebook#5927)
  Add note to docs about using Sass and Flow together (facebook#5823)
  Update PWA link in README (facebook#5907)
  Add placeholders to template README for bit.ly links. (facebook#5808)
  Disable copy to clipboard in cra --info (facebook#5905)
  Support setupTests.ts (facebook#5698)
  Remove unnecessary whitespace in template HTML
  Run prettier on HTML files (facebook#5839)
  Some Grammar fixes (facebook#5858)
  Fix link to page about running tests (facebook#5883)
  fix: make typescriptformatter support 0.5 of fork checker (facebook#5879)
  ...
@lock lock bot locked and limited conversation to collaborators Jan 18, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

4 participants
You can’t perform that action at this time.