-
Notifications
You must be signed in to change notification settings - Fork 2
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
ab-core uses eslint 9 (take 2) #1696
Conversation
# Conflicts: # libs/@guardian/browserslist-config/package.json # pnpm-lock.yaml
`source` has no peerdep on `libs`, so we can use the local copy
… into sndrs/use-eslint9-config # Conflicts: # libs/@guardian/ab-react/package.json # libs/@guardian/core-web-vitals/package.json # libs/@guardian/identity-auth/package.json # libs/@guardian/source-development-kitchen/.storybook/main.ts # libs/@guardian/source-development-kitchen/package.json # libs/@guardian/source/.storybook/main.ts # libs/@guardian/source/package.json # pnpm-lock.yaml
|
Tip Once this PR is ready to go, add the This saves us a lot of money by not running the tests before we need them. |
Co-authored-by: Oliver Abrahams <[email protected]>
…9-config # Conflicts: # libs/@guardian/core-web-vitals/package.json # libs/@guardian/source-development-kitchen/package.json # libs/@guardian/source/package.json # pnpm-lock.yaml
this is 2nd go at #1668, which was accidentally auto-merged into another feature branch...
What are you changing?
@guardian/eslint-config
to support ESLint v9 #1658@guardian/eslint-config
slightly to re-enable comments lintingWhy?
Anything else?
yeah - noticed doing this that we don't lint anything outside
libs/**
, so will add that in a further PR