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

feat: add a defineConfig utility function and use it in examples #117

Merged
merged 1 commit into from
Dec 25, 2024

Conversation

haoqunjiang
Copy link
Member

@haoqunjiang haoqunjiang commented Dec 14, 2024

This commit also exports the createConfig as a named export for consistency in import styles.
I feel that though the name is descriptive enough, it's not a suitable name to be used directly in the defineConfig arguments, so I used import aliases in all examples.

typescript-eslint 8.15 added support for infinitely deep array nesting in the config helper, so I removed all the spread patterns in the examples. typescript-eslint/typescript-eslint#10333

As discussed in vuejs/create-eslint-config#37

/cc @segevfiner

This commit also exports the `createConfig` as a named export for
consistency.
@haoqunjiang
Copy link
Member Author

Gonna need it in my @vue/eslint-config-standard-with-typescript refactor so I'm merging it.

@haoqunjiang haoqunjiang merged commit 3c930e4 into main Dec 25, 2024
6 checks passed
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.

1 participant