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

Add support for eslint flat config #2870

Closed
wants to merge 4 commits into from
Closed

Add support for eslint flat config #2870

wants to merge 4 commits into from

Conversation

OlivierZal
Copy link

No description provided.

@OlivierZal
Copy link
Author

@ljharb, here is the PR to make airbnb/javascript#2818 work.

config/flat.js Outdated Show resolved Hide resolved
@ljharb
Copy link
Member

ljharb commented Aug 31, 2023

this may also need to be coordinated with #2829.

Co-authored-by: Jordan Harband <[email protected]>
@OlivierZal OlivierZal requested a review from ljharb August 31, 2023 23:25
@ljharb
Copy link
Member

ljharb commented Sep 1, 2023

@bmish this PR adds a new entry point; what needs to be done for the docs generator to recognize it?

@ljharb
Copy link
Member

ljharb commented Sep 2, 2023

hm, maybe it's not yet compatible with flat config?

@OlivierZal OlivierZal closed this Sep 2, 2023
@OlivierZal OlivierZal deleted the olivierzal/import_plugin/eslint_flat_config_support branch September 2, 2023 11:52
@ljharb
Copy link
Member

ljharb commented Sep 2, 2023

um, I’m confused, why did you delete your forks, making these PRs unrecoverable??

@bmish
Copy link
Contributor

bmish commented Sep 2, 2023

I have an issue related to this: bmish/eslint-doc-generator#376. Any thoughts on what needs to be done for eslint-doc-generator to support flat config?

@ljharb
Copy link
Member

ljharb commented Sep 2, 2023

Seems like it relies on the plugin name in an entry point being a string.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants