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

Bump fork-ts-checker-webpack-plugin and notifier, update from babel-eslint to babel/eslint-parser #1985

Merged

Conversation

daveajrussell
Copy link
Contributor

@daveajrussell daveajrussell commented Oct 29, 2024

Following #1983, a typeerror began emanating from the fork-ts-checker-webpack-plugin package during builds, due to the bump in the version of eslint. Whilst builds were still successfully producing artefacts, it's possible we'd no longer be notified of ts issues.

This updates fork-ts-checker-webpack-plugin to version 6.5.3.. version 7.0.0 and onward drop support for webpack 4 and eslint, which would require a much larger change. Also updated is the fork-ts-checker-notifier-webpack-plugin package to version 4.0.0.

The update to fork-ts-checker-webpack-plugin necessitated an upgrade from babel-eslint to @babel/eslint-parser due to an error emanating from eslint-scope during builds. The only necessary change from this was to update our eslint config to use @babel/eslint-parser instead.

@daveajrussell daveajrussell merged commit 323e84c into neo4j:master Oct 30, 2024
16 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.

2 participants