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

Use flat config for eslint 9 #608

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

SimonSimCity
Copy link

@SimonSimCity SimonSimCity commented Feb 19, 2025

If I do not use this, I get the error #508, so I guess the wrong config is referenced in the getting started guide.

I'm using this package @intlify/eslint-plugin-vue-i18n in version 3.2.0.

Maybe good to note in addition: I'm not using eslint-plugin-vue, because I want to use some of your linting things in a React environment. Specially your support of tsx files, and the extensive list of rules that are available, brought me to this idea.

Copy link

changeset-bot bot commented Feb 19, 2025

⚠️ No Changeset found

Latest commit: 74ca2d3

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@ota-meshi
Copy link
Member

Oh, we're working on v4 and the current guide is for the upcoming v4.
So for now we might need to document how to configure it for v3 and v4.
Can you do that?

@SimonSimCity
Copy link
Author

@ota-meshi Is it fine if I just add a note for those who want to try out the new version 4? I'd not put it as part of the base example, as I'd rather keep it to the latest stable version.

@ota-meshi
Copy link
Member

I would like to see what the note is like, so could you please commit the note once?

@ota-meshi
Copy link
Member

Thanks for the change.
However, all current documentation is written for v4, so rather than writing only a portion of it for v3, I think it makes more sense to explain the configuration for v4 and then direct you to the documentation for v3.
https://github.com/intlify/eslint-plugin-vue-i18n/blob/v3.2.0/docs/started.md

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

Successfully merging this pull request may close these issues.

2 participants