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

chore: format/lint .cjs and .mjs files #376

Merged
merged 1 commit into from
Apr 23, 2024
Merged

Conversation

matijs
Copy link
Member

@matijs matijs commented Mar 26, 2024

Make sure Prettier, ESLint and lint-staged know about .cjs and .mjs files

@matijs matijs requested a review from Robbert March 26, 2024 12:56
@matijs matijs enabled auto-merge (rebase) March 26, 2024 12:56
@matijs matijs disabled auto-merge March 26, 2024 12:56
@matijs matijs enabled auto-merge (squash) March 26, 2024 12:56
@matijs matijs disabled auto-merge March 26, 2024 13:01
@matijs matijs changed the title chore: format/lint .cjs files chore: format/lint .cjs and .mjs files Mar 26, 2024
@matijs
Copy link
Member Author

matijs commented Mar 26, 2024

@Robbert this adds quite a bit of config just for parsing an import assertion. We could also get rid of the assertion by using Node's readFile from node:fs/promises and using JSON.parse.

Robbert
Robbert previously approved these changes Apr 9, 2024
.eslintrc.json Show resolved Hide resolved
@matijs matijs mentioned this pull request Apr 9, 2024
Make sure Prettier, ESLint and lint-staged know about .cjs and .mjs files

Use `@babel/eslint-parser` to parse .cjs and .mjs files and allow the
deprecated import assertions syntax (`assert`) for now.
@matijs matijs enabled auto-merge (rebase) April 12, 2024 11:50
@matijs matijs disabled auto-merge April 12, 2024 11:50
@matijs matijs enabled auto-merge (rebase) April 12, 2024 11:50
@matijs matijs requested a review from Robbert April 12, 2024 11:50
@matijs matijs requested review from bddjong and removed request for Robbert April 19, 2024 12:04
@matijs matijs merged commit 679b412 into main Apr 23, 2024
2 checks passed
@matijs matijs deleted the chore/deal-with-cjs-files branch April 23, 2024 14:33
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.

3 participants