philihp / week-date / 25929668808

100%

master: 100%

Pull #34

github

Modernize tooling and drop the build step

This is a 35-line library; the babel transpile chain, copy-pasted
frontend jest config, and husky/lint-staged setup were heavier than
the code they wrapped. Rewrite the source as plain CommonJS and ship
src/ directly so consumers (and tests) just require it.

- Source: ESM `export` → `module.exports`; tests use `require`.
- Delete .babelrc.js; drop @babel/cli, @babel/core, @babel/preset-env.
- Drop the `build` and `prepare` scripts; point `main` at
  ./src/index.js and add a `files` allowlist so npm publishes just src/.
- Drop husky + lint-staged config (optional local-only tooling that
  the maintainer can re-add at will).
- Delete jest.config.js — its only non-default settings were a
  frontend-app asset moduleNameMapper and a babel-jest transform,
  neither of which apply to a CommonJS date library.
- Bump remaining devDeps: jest 26 → 30, prettier 2 → 3,
  @philihp/eslint-config 3 → 6 (eslint 7 → 8).
- CI: bump Node matrix from 10/11/12/13/14.x (all EOL) to 18/20/22.x;
  bump actions/{checkout,setup-node} to v4 and coverallsapp to v2;
  drop the now-meaningless `npm run build --if-present` step.
- Bump .tool-versions to nodejs 22.11.0.

All 22 tests pass; coverage still produces coverage/lcov.info for
Coveralls.

3 of 3 branches covered (100.0%)

Branch coverage included in aggregate %.

2 of 2 new or added lines in 1 file covered. (100.0%)

19 of 19 relevant lines covered (100.0%)

24.11 hits per line