Skip to content

Update README#33

Merged
wjt merged 3 commits intomainfrom
push-xluypnlrmzox
Jun 20, 2025
Merged

Update README#33
wjt merged 3 commits intomainfrom
push-xluypnlrmzox

Conversation

@wjt
Copy link
Copy Markdown
Member

@wjt wjt commented Jun 20, 2025

No description provided.

wjt added 3 commits June 20, 2025 13:57
Previously, the recommended concurrency settings for the publish
workflow were in a separate block. In hindsight I think this makes the
documentation harder to follow: it is easier to have a single example
you can copy to your project.
The limitation around merged branches still being present in the
amalgamated site has been solved by a combination of commit
4fe4270 (which ignores branches which
correspond to a closed pull request), commit
bacf009 (which ignores artifacts for
deleted branches), and documenting the recommendation to run the publish
workflow when a branch is deleted.

Remove this caveat, and spell out the expired-artifact caveat more
clearly.
@wjt wjt requested a review from cassidyjames June 20, 2025 13:18
Copy link
Copy Markdown

@cassidyjames cassidyjames left a comment

Choose a reason for hiding this comment

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

This reads well to me!

@wjt wjt merged commit 1589fb1 into main Jun 20, 2025
1 check passed
@wjt wjt deleted the push-xluypnlrmzox branch June 20, 2025 21:23
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.

2 participants