Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

ci: change docs build to use yarn #6700

Merged
merged 3 commits into from
Feb 19, 2025
Merged

ci: change docs build to use yarn #6700

merged 3 commits into from
Feb 19, 2025

Conversation

mendonk
Copy link
Collaborator

@mendonk mendonk commented Feb 18, 2025

This pull request updates the deployment workflow for GitHub Pages in the .github/workflows/deploy_gh-pages.yml file. The primary change is switching the package manager from npm to yarn. This workflow currently fails because it's looking for package-lock.json.

Also sync dependencies.

Build test succeeds

@dosubot dosubot bot added the size:XS This PR changes 0-9 lines, ignoring generated files. label Feb 18, 2025
@mendonk mendonk self-assigned this Feb 18, 2025
@github-actions github-actions bot added the documentation Improvements or additions to documentation label Feb 18, 2025
@dosubot dosubot bot added size:S This PR changes 10-29 lines, ignoring generated files. and removed size:XS This PR changes 0-9 lines, ignoring generated files. labels Feb 18, 2025
@github-actions github-actions bot added documentation Improvements or additions to documentation and removed documentation Improvements or additions to documentation labels Feb 18, 2025
@mendonk mendonk requested a review from ogabrielluiz February 18, 2025 22:06
@github-actions github-actions bot added documentation Improvements or additions to documentation and removed documentation Improvements or additions to documentation labels Feb 18, 2025
@mendonk mendonk changed the title docs: change build to use yarn ci: change docs build to use yarn Feb 19, 2025
Copy link
Contributor

@ogabrielluiz ogabrielluiz left a comment

Choose a reason for hiding this comment

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

LGTM

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Feb 19, 2025
@ogabrielluiz ogabrielluiz added this pull request to the merge queue Feb 19, 2025
Merged via the queue into main with commit 770c1b3 Feb 19, 2025
27 of 36 checks passed
@ogabrielluiz ogabrielluiz deleted the docs-packagelock branch February 19, 2025 14:33
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
documentation Improvements or additions to documentation lgtm This PR has been approved by a maintainer size:S This PR changes 10-29 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants