Skip to content

fix(ci): publish to npm in its own job; remove duplicate Pages workflow - #45

Merged
codinglombok merged 2 commits into
mainfrom
claude/affectionate-sagan-o4rxmx
Sep 30, 2026
Merged

codinglombok merged 2 commits into
mainfrom
claude/affectionate-sagan-o4rxmx

Conversation

@codinglombok

@codinglombok codinglombok commented Sep 30, 2026 •

Copy link
Copy Markdown
Owner

Summary

  • release-please.yml: npmjs publish moves to its own publish-npm job. Both publish jobs depend only on release-please, so an npm authentication failure no longer skips the GitHub Packages publish.
  • Remove .github/workflows/static.yml (second Pages deployment of the repository root) and untrack CLAUDE.md.

Type

  • Bug fix
  • Chore / tooling

Checklist

  • yamllint and checkov pass
  • No source changes; bundles unchanged

In the 0.1.10 release the npmjs publish step failed (E404 on PUT, the
NPM_TOKEN no longer has publish rights). Because that step lived in the
release-please job, publish-github-packages (needs: release-please) was
skipped too, so no registry got 0.1.10.

Move the npm publish into a separate publish-npm job that, like the
GitHub Packages job, only needs release-please. An npm auth problem now
fails just that job.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_011pv5NUFsXGzyv4F9TcK7Rz
Merge d89c64a ("Merge branch 'main' of https://github.com/codinglombok/LombokCSS")
joined main with the pre-cleanup history (44f0ef5) and brought back two
files that had been removed:

- .github/workflows/static.yml, the second Pages deployment that uploads
  the repo root (no index.html) and can take the site down with a 404
  (removed in #42)
- CLAUDE.md, internal Claude Code memory that must stay local
  (untracked in #41; still listed in .gitignore)

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_011pv5NUFsXGzyv4F9TcK7Rz
@github-actions github-actions Bot added documentation Improvements or additions to documentation ci labels Sep 30, 2026
@github-actions

Copy link
Copy Markdown
Contributor

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Scanned Files

  • .github/workflows/static.yml

@github-actions

Copy link
Copy Markdown
Contributor

Message that will be displayed on users' first pull request

Copy link
Copy Markdown
Owner Author

CI note on eb7ae68: github-advanced-security failed, and not because of this PR. The Copilot autofind job exits before analyzing any code with CAPIError: 400 The requested model is not supported — the same failure reported on #36 and #38 (it passed on #44, so it is intermittent on the Copilot side). No fix exists in this repo's code. A re-run was attempted but GitHub refuses it for this run (403 This workflow run cannot be retried); the Copilot model configured for that check needs to be updated by the repo owner.


Generated by Claude Code

@codinglombok
codinglombok merged commit c2de5fa into main Sep 30, 2026
27 of 28 checks passed
@codinglombok
codinglombok deleted the claude/affectionate-sagan-o4rxmx branch September 30, 2026 23:37
@codinglombok codinglombok changed the title fix(ci): decouple npm publish from GitHub Packages; drop files re-added by an old-clone merge fix(ci): publish to npm in its own job; remove duplicate Pages workflow Oct 1, 2026
codinglombok added a commit that referenced this pull request Oct 1, 2026
…rxmx

fix(ci): decouple npm publish from GitHub Packages; drop files re-added by an old-clone merge
codinglombok added a commit that referenced this pull request Oct 3, 2026
…rxmx

fix(ci): decouple npm publish from GitHub Packages; drop files re-added by an old-clone merge
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants