viiccwen opened a new pull request, #1426:
URL: https://github.com/apache/mahout/pull/1426
### Related Issues
Closes #1353
### Changes
- [ ] Bug fix
- [ ] New feature
- [ ] Refactoring
- [ ] Documentation
- [ ] Test
- [x] CI/CD pipeline
- [ ] Other
### Why
The website build check already validates the Docusaurus build, and `npm run
build` already generates the Python API docs through the website `prebuild`
script. However, the workflow was only triggered by website/docs path changes,
so Python API source changes that affect generated API documentation did not
necessarily run the website validation check.
The API docs validation workflow should stay read-only and avoid
deploy/write behavior.
### How
- Added Python API source paths to the existing website build check trigger:
- `qumat/**`
- `qdp/qdp-python/qumat_qdp/**`
- Set explicit `contents: read` permissions for the validation workflow.
- Kept the existing website build behavior unchanged; `npm run build`
continues to generate API docs via `prebuild`.
## Checklist
- [x] Added or updated unit tests for all changes (not applicable:
workflow-only change; verified with `actionlint` and website build)
- [x] Added or updated documentation for all changes (not applicable:
workflow trigger update only)
Verification:
- `actionlint .github/workflows/website-build.yml`
- `git diff --check`
- `npm run typecheck && npm run build`
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
