Skip to content

Comments

Remove leftover commonjs directives; update example#28

Merged
mattcg merged 1 commit intomattcg:masterfrom
Fdawgs:chore/cjs
Mar 6, 2025
Merged

Remove leftover commonjs directives; update example#28
mattcg merged 1 commit intomattcg:masterfrom
Fdawgs:chore/cjs

Conversation

@Fdawgs
Copy link
Contributor

@Fdawgs Fdawgs commented Mar 6, 2025

ESM modules are strict by default, so the 'use strict' directives are no longer needed.

Likewise, I've updated the example in the readme to use an ESM import rather than a cjs require.

@Fdawgs Fdawgs changed the title chore: remove leftover commonjs directives and example chore: remove leftover commonjs directives; update example Mar 6, 2025
@Fdawgs Fdawgs changed the title chore: remove leftover commonjs directives; update example Remove leftover commonjs directives; update example Mar 6, 2025
@coveralls
Copy link

Coverage Status

coverage: 94.411% (-0.05%) from 94.461%
when pulling 852afc3 on Fdawgs:chore/cjs
into aba012d on mattcg:master.

@mattcg mattcg merged commit 5909c55 into mattcg:master Mar 6, 2025
9 of 14 checks passed
@mattcg
Copy link
Owner

mattcg commented Mar 6, 2025

Thank you, Frazer

@Fdawgs Fdawgs deleted the chore/cjs branch March 6, 2025 10:35
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