Skip to content

Misc cleanup: bump auto-assign action, autoprefixer/postcss-cli, stop tracking generated cli-docs - #744

Open
rossigee wants to merge 1 commit into
goharbor:mainfrom
rossigee:split/misc-cleanup
Open

Misc cleanup: bump auto-assign action, autoprefixer/postcss-cli, stop tracking generated cli-docs#744
rossigee wants to merge 1 commit into
goharbor:mainfrom
rossigee:split/misc-cleanup

Conversation

@rossigee

Copy link
Copy Markdown

Summary

Split out from #731 per review feedback, as the lowest-risk, independent piece. This does not depend on the other two split-out PRs.

  • Bump kentaro-m/auto-assign-action v1.2.4 -> v2.1.2
  • Bump autoprefixer 9.7.4 -> 10.5.2 and postcss-cli 7.1.2 -> 11.0.1 (fixes the picomatch ReDoS npm audit advisory)
  • content/cli-docs/ is regenerated on every build by cli-docs.sh but was committed to git, causing spurious diffs each run. Added to .gitignore and untracked the 109 already-committed files.

Related

Part of the Hugo/Bulma upgrade work originally proposed in #731, split into independently reviewable pieces per @OrlinVasilev's request there. See also the other two parts:

  • Hugo 0.163.3 + Bulma 1.0.4/Dart Sass migration (to follow)
  • Playwright E2E suite + CI workflow (to follow, depends on the above)

Test plan

  • npm audit reports 0 vulnerabilities after the bump
  • postcss-cli 11 + autoprefixer 10 verified to process CSS correctly (existing postcss.config.js unchanged)

… tracking generated cli-docs

- kentaro-m/auto-assign-action v1.2.4 -> v2.1.2
- autoprefixer 9.7.4 -> 10.5.2, postcss-cli 7.1.2 -> 11.0.1 (fixes
  npm audit picomatch ReDoS advisory)
- content/cli-docs/ is regenerated on every build by cli-docs.sh but
  was committed to git, causing spurious diffs each time it runs.
  Add it to .gitignore and untrack the 109 already-committed files.

Verified: postcss-cli 11 + autoprefixer 10 process CSS correctly
(existing postcss.config.js unchanged), npm audit reports zero
vulnerabilities after the bump.

Signed-off-by: Ross Golder <ross@golder.org>
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.

4 participants