Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Limit the publish_website.yml work-flow to only the master branch (issue 16369) #16370

Merged

Conversation

Snuffleupagus
Copy link
Collaborator

@Snuffleupagus Snuffleupagus changed the title Limit the publish_website.yml work-flow to only the master branch (issue 16369) Limit the publish_website.yml work-flow to only the master branch (issue 16369) Apr 28, 2023
@Snuffleupagus Snuffleupagus force-pushed the publish-website-only-master branch from 4c302e1 to 09edd66 Compare April 28, 2023 08:32
@Snuffleupagus
Copy link
Collaborator Author

Looking at https://github.com/Snuffleupagus/pdf.js/actions it does appear that this prevented the publish_website.yml work-flow from running against this patch, which was indeed the intended result.
Whether it actually runs when the patch is merged is another question :-)

@timvandermeij timvandermeij merged commit 797f8d3 into mozilla:master Apr 29, 2023
@timvandermeij
Copy link
Contributor

Yes, this sounds like a very good idea. Thank you for finetuning this!

@Snuffleupagus Snuffleupagus deleted the publish-website-only-master branch April 29, 2023 09:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Limit the publish_website.yml work-flow to only the master branch
2 participants