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

Allow CI workflow to be triggered in different ways. #25

Closed
wants to merge 1 commit into from

Conversation

advaitjain
Copy link
Member

@advaitjain advaitjain commented Apr 26, 2021

  • from pull request, if the appropriate label is applied
  • manual triggering via workflow_dispatch
  • part of a scheduled build

Prior to this change, manually triggering the CI workflow, or a scheduled run would be skipped because the ci:run label was not found.

 * from pull request, if the appropriate label is applied
 * manual triggering via workflow_dispatch
 * part of a scheduled build
@advaitjain advaitjain changed the title Attempting to be able to run CI workflow from PR, manually, and on schedule. Allow CI workflow to be triggered in different ways. Apr 26, 2021
@advaitjain
Copy link
Member Author

closing this PR in favor of #26

@advaitjain advaitjain closed this Apr 26, 2021
advaitjain added a commit that referenced this pull request Apr 26, 2021
With #25 we went down a path that appeared at first to allow for the checks to be consolidated in an environment variable but ended up being quite complex.

This PR instead modifies the per-job if check to either have the ci:run label or originate not from a pull request.
@advaitjain advaitjain deleted the ci-on-schedule branch April 26, 2021 23:55
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.

1 participant