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

ESLint v9 #81

Closed
wants to merge 2 commits into from
Closed

ESLint v9 #81

wants to merge 2 commits into from

Conversation

danbillson
Copy link
Contributor

The key part of this change is the addition of this eslint rule:

'import/extensions': ['error', 'always', { js: 'always' }],

which enforces that all imports and exports contain the .js extension as this can easily be missed and can break certain builds as seen in #76

Other updates

  • rename tsconfig.base.json to tsconfig.json - this was required for the eslint upgrade which couldn't find a root tsconfig

@danbillson danbillson self-assigned this Nov 26, 2024
@danbillson danbillson closed this Nov 26, 2024
@vijayasingam-paddle vijayasingam-paddle deleted the file-imports branch January 2, 2025 15:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant