Sponsored Content
Skip to content

ci: Use latest version of publish-docs workflow - #6824

Merged
bthomee merged 1 commit into
XRPLF:ripple/staging-313from
mathbunnyru:update_publish_docs
Apr 9, 2026
Merged

ci: Use latest version of publish-docs workflow#6824
bthomee merged 1 commit into
XRPLF:ripple/staging-313from
mathbunnyru:update_publish_docs

Conversation

@mathbunnyru

Copy link
Copy Markdown
Contributor

High Level Overview of Change

I've taken the latest version of this workflow from develop, should work fine even for old branch.

Context of Change

API Impact

  • Public API: New feature (new methods and/or new fields)
  • Public API: Breaking change (in general, breaking changes should only impact the next api_version)
  • libxrpl change (any change that may affect libxrpl or dependents of libxrpl)
  • Peer protocol change (must be backward compatible or bump the peer protocol version)

@mathbunnyru
mathbunnyru requested a review from bthomee April 9, 2026 14:21
@mathbunnyru mathbunnyru added Trivial Simple change with minimal effect, or already tested. Only needs one approval. Ready to merge *PR author* thinks it's ready to merge. Has passed code review. Perf sign-off may still be required. labels Apr 9, 2026
Comment on lines +37 to +39
# ubuntu-latest has only 2 CPUs for private repositories
# https://docs.github.com/en/actions/reference/runners/github-hosted-runners#standard-github-hosted-runners-for--private-repositories
NPROC_SUBTRACT: ${{ github.event.repository.visibility == 'public' && '2' || '1' }}

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

It might make also/more sense to update the action to use a minimum of 1 processor. (not now and not here though)

@bthomee
bthomee merged commit fde03c8 into XRPLF:ripple/staging-313 Apr 9, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Ready to merge *PR author* thinks it's ready to merge. Has passed code review. Perf sign-off may still be required. Trivial Simple change with minimal effect, or already tested. Only needs one approval.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants