Skip to content

Conversation

jonkoops
Copy link
Contributor

By submitting a PR to this repository, you agree to the terms within the Auth0 Code of Conduct. Please see the contributing guidelines for how to create and submit a high-quality PR for this repo.

Description

Upgrades the linting tooling (ESLint, Prettier, publint), and fixes the resulting warnings/errors. The ESlint configuration has been migrated to the new 'flat' configuration format required by ESLint v9. ESLint is now also separated from Prettier into different scripts, as recommended by the Prettier documentation on linter integration.

Prettier will now format all types of files known to it, and not just the

References

@jonkoops jonkoops requested a review from a team as a code owner September 10, 2025 14:08
Comment on lines -7 to -8
"plugin:import/recommended",
"plugin:import/typescript",
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The import plugin has been removed, as it has known compatibility issues with ESLint v9, and there does not seem an intent to fix this anytime soon. Considering the imports are already checked by the TypeScript compiler, I have chosen to omit it.

Comment on lines +22 to +23
"@typescript-eslint/no-unnecessary-type-parameters": "off",
"@typescript-eslint/restrict-template-expressions": "off",
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These rules seem to give some false positives, so I have disabled them.

Signed-off-by: Jon Koops <jonkoops@gmail.com>
@jonkoops
Copy link
Contributor Author

@frederikprijck could I ask you to take a look at this one? I'll make some other PRs as well to get all the dependencies upgraded to their respective latest versions.

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