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

Add .cjs extension for JavaScript #3866

Merged
merged 1 commit into from May 19, 2024
Merged

Add .cjs extension for JavaScript #3866

merged 1 commit into from May 19, 2024

Conversation

Chocobo1
Copy link
Contributor

@Chocobo1 Chocobo1 commented May 5, 2024

.cjs is a commonly used file extension for Node.js and it is the complementary of .mjs.
See: https://nodejs.org/docs/latest/api/esm.html#enabling

`.cjs` is a commonly used file extension for Node.js and it is the
complementary of `.mjs`.
See: https://nodejs.org/docs/latest/api/esm.html#enabling
@elextr
Copy link
Member

elextr commented May 5, 2024

LGTM, doesn't clash with any other extensions, if no objections can be merged in about a week.

@elextr elextr merged commit 529f5d4 into geany:master May 19, 2024
4 checks passed
@Chocobo1 Chocobo1 deleted the cjs branch May 19, 2024 05:24
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.

None yet

2 participants