Skip to content

ci(docs): deploy docs to Netlify, bump pnpm/action-setup to v6 - #2

Merged
luishcastroc merged 3 commits into
mainfrom
netlify-docs-deploy
Aug 14, 2026
Merged

ci(docs): deploy docs to Netlify, bump pnpm/action-setup to v6#2
luishcastroc merged 3 commits into
mainfrom
netlify-docs-deploy

Conversation

@luishcastroc

Copy link
Copy Markdown
Contributor

What

  • Rewrites the docs deploy workflow to publish to Netlify via netlify-cli instead of GitHub Pages, which was failing with a 404 (Ensure GitHub Pages has been enabled) since Pages is not enabled on this repo. Drops the Pages-specific pieces: pages/id-token permissions, the deploy-pages job, the 404.html SPA fallback copy, and the DOCS_BASE=/angular-gsap/ base path (Netlify serves from the domain root).
  • Adds apps/docs/public/_redirects (/* /index.html 200) as the Netlify SPA fallback; prerendered pages still win because real files are served before redirects.
  • Bumps pnpm/action-setup v4 → v6 in both workflows: v5 moved the action to Node 24 (clears the Node 20 deprecation warning), v6 added pnpm 11 support, which this repo uses.
  • Docs fact-check fixes: home page tree-shaking card said 9 other directives (there are now 12 total), README size claim now says what was actually measured (~6 kB min+gzip), and the reference page gains the missing effects row for provideGsap.

Setup required before the deploy works

The workflow needs two repo secrets: NETLIFY_AUTH_TOKEN and NETLIFY_SITE_ID. Until they exist the deploy step will fail on auth.

🤖 Generated with Claude Code

luishcastroc and others added 3 commits August 14, 2026 08:38
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@luishcastroc
luishcastroc merged commit b142fbb into main Aug 14, 2026
1 check passed
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.

1 participant