CommonJS --> ESM

[chris48s]

2 problems to solve

1. Actions can't run on node14+ yet: Supporting Node.js 14 and 16 LTS as the runtime of JavaScript action actions/runner#772
2. Adopt top-level await. Not done yet - still using (async () => {.... Eslint doesn't yet allow it
3. nyc coverage doesn't work. Tests all run/pass but the cov report is

---------------|---------|----------|---------|---------|-------------------
File           | % Stmts | % Branch | % Funcs | % Lines | Uncovered Line #s 
---------------|---------|----------|---------|---------|-------------------
All files      |       0 |        0 |       0 |       0 |                   
 action.js     |       0 |      100 |       0 |       0 | 4-5               
 formatters.js |       0 |        0 |       0 |       0 | 3-24              
 lib.js        |       0 |        0 |       0 |       0 | 8-96              
---------------|---------|----------|---------|---------|-------------------

[codecov-commenter]

Codecov Report

Attention: 3 lines in your changes are missing coverage. Please review.

Comparison is base (f53fdb3) 72.72% compared to head (627ffa8) 86.66%.
Report is 1 commits behind head on main.

Additional details and impacted files

@@             Coverage Diff             @@
##             main      #16       +/-   ##
===========================================
+ Coverage   72.72%   86.66%   +13.93%     
===========================================
  Files           3        3               
  Lines          66      135       +69     
  Branches        7       28       +21     
===========================================
+ Hits           48      117       +69     
- Misses         17       18        +1     
+ Partials        1        0        -1     

Files	Coverage Δ
src/formatters.js	92.59% <100.00%> (+5.92%)	⬆️
src/lib.js	88.46% <100.00%> (+13.99%)	⬆️
src/action.js	0.00% <0.00%> (ø)

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[chris48s]

If there is not an updated version of nyc by the time GH Actions allows node>12 maybe the solution is to switch to https://github.com/bcoe/c8 for coverage

[chris48s]

Still to do:

• nyc --> c8
• ncc now spits out https://github.com/action-badges/package-json-badges/blob/332eb0ae56096d00cd29461506e0ab5441a50a85/dist/package.json into the dist directory. Check if the new dist works, or do I need to fiddle with it?

[chris48s]

Suspect the dist is fine. I've got the same over on https://github.com/chris48s/satisfying-python/tree/main/dist

[socket-security]

New and removed dependencies detected. Learn more about Socket for GitHub ↗︎

Packages	Version	New capabilities	Transitives	Size	Publisher
c8	8.0.1	filesystem, environment	+14	1.18 MB	oss-bot

🚮 Removed packages: [email protected]