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

[readme] Fix shared settings link #3834

Merged

Conversation

MgenGlder
Copy link
Contributor

@MgenGlder MgenGlder commented Sep 29, 2024

Summary

  • Updates the link to the eslint shared settings page in the README.md file.

As I was upgrading my eslint configuration for a personal project, I found that I needed to use the shared settings feature to configure my React version. The link in the docs section navigates to the main page but doesn't anchor you to the section that discusses the shared settings. I think the original link broke at some point. This PR fixes it.

Copy link
Member

@ljharb ljharb left a comment

Choose a reason for hiding this comment

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

Thanks! Sad that the link broke, cool URLs don't change :-(

@ljharb ljharb force-pushed the mgenglder/fix-shared-settings-link branch from a4d4c25 to 1f95a24 Compare September 30, 2024 00:44
@ljharb ljharb changed the title docs: Fix shared settings link in README.md example [readme] Fix shared settings link Sep 30, 2024
@ljharb ljharb merged commit 1f95a24 into jsx-eslint:master Sep 30, 2024
376 checks passed
renovate bot added a commit to mmkal/eslint-plugin-mmkal that referenced this pull request Oct 1, 2024
github-merge-queue bot pushed a commit to zemn-me/monorepo that referenced this pull request Oct 1, 2024
github-merge-queue bot pushed a commit to zemn-me/monorepo that referenced this pull request Oct 1, 2024
renovate bot added a commit to mmkal/eslint-plugin-mmkal that referenced this pull request Oct 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

2 participants