Skip to content

feat(create-vite): bump eslint-plugin-react-refresh#21534

Open
ArnaudBarre wants to merge 1 commit intomainfrom
arnaud/bump-eslint-react-refresh
Open

feat(create-vite): bump eslint-plugin-react-refresh#21534
ArnaudBarre wants to merge 1 commit intomainfrom
arnaud/bump-eslint-react-refresh

Conversation

@ArnaudBarre
Copy link
Member

@ArnaudBarre ArnaudBarre commented Feb 1, 2026

I did a major bump to my eslint plugin for multiple reasons.

The configs are now functions which allow users to merge additional settings with the default settings. But ESLint typing forbids this to be on <default export>.config, so this why a named export it used

@ArnaudBarre ArnaudBarre self-assigned this Feb 1, 2026
@ArnaudBarre ArnaudBarre changed the title deps(create-vite): bump eslint-plugin-react-refresh feat(create-vite): bump eslint-plugin-react-refresh Feb 1, 2026
@bluwy
Copy link
Member

bluwy commented Feb 2, 2026

The configs are now functions which allow users to merge additional settings with the default settings. But ESLint typing forbids this to be on .config, so this why I need to

What about exporting the default options object from the plugin so that users can manually spread themselves?

@ArnaudBarre
Copy link
Member Author

Yeah I could also export this. But function are nice because parameters gets TS autocomplete which makes options discoverable, contrary to the default way of configuring ESLint

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