Skip to content

ci: add docs build workflow#3235

Merged
reynaldichernando merged 3 commits into
HeyPuter:mainfrom
wwb1942:ci/docs-build
Jun 10, 2026
Merged

ci: add docs build workflow#3235
reynaldichernando merged 3 commits into
HeyPuter:mainfrom
wwb1942:ci/docs-build

Conversation

@wwb1942

@wwb1942 wwb1942 commented Jun 9, 2026

Copy link
Copy Markdown
Contributor

Closes #3225.

Adds a CI workflow that runs when documentation files under src/docs/** change.

Changes:

  • Triggers on PRs touching src/docs/**
  • Uses Node 24
  • Runs npm ci at repo root
  • Runs npm run build in src/docs

Verified locally:

  • workflow YAML parses
  • npm run build passes in src/docs under Node 24/npm 11

wwb1942 added 2 commits June 9, 2026 18:56
Add CI workflow that runs npm ci and npm run build in src/docs

whenever files under src/docs change in a pull request.

Closes HeyPuter#3225
@CLAassistant

CLAassistant commented Jun 9, 2026

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

Comment thread .github/workflows/docs-build.yml Outdated
Comment thread .github/workflows/docs-build.yml Outdated
@reynaldichernando

Copy link
Copy Markdown
Member

hi @wwb1942 , thank you for your contribution, i've left some review for the changes!

@wwb1942

wwb1942 commented Jun 10, 2026

Copy link
Copy Markdown
Contributor Author

hi @wwb1942 , thank you for your contribution, i've left some review for the changes!

Addressed the review in 55d2a66:

  • removed the workflow file from pull_request.paths
  • removed the explicit permissions block

Verified locally that the workflow YAML parses and npm run build passes in src/docs under Node 24/npm 11.

@reynaldichernando reynaldichernando left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@reynaldichernando reynaldichernando merged commit 2dd2887 into HeyPuter:main Jun 10, 2026
4 checks 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.

Add CI for docs changes

3 participants