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

[pull] master from john-kurkowski:master #53

Merged
merged 1 commit into from
Mar 7, 2024

Conversation

pull[bot]
Copy link

@pull pull bot commented Mar 6, 2024

See Commits and Changes for more details.


Created by pull[bot]

Can you help keep this open source service alive? 💖 Please sponsor : )

This is a minor optimization of the following.

1. Only run the workflow 1x instead of 2x on a PR that's from a branch
   instead of a fork.
    * This is done by only triggering on push events when they're to the
      master branch. If you want to trigger the workflow on other
      branches, be sure to open a PR.
2. Do not run the workflow on tags.
    * Tags are made on existing commits on the master branch, which
      already has workflow coverage, per the previous point.

Optimizing this isn't a big deal for an open source project on GitHub,
where GitHub Actions usage is free. However, it is a big deal for
private repository development, where usage is limited (where I've been
doing some testing).
@pull pull bot added the ⤵️ pull label Mar 6, 2024
@pull pull bot merged commit 6632577 into ddelange:master Mar 7, 2024
27 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant