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

Support user-provided base/head refs & non-PR workflows #165

Merged
merged 2 commits into from
Aug 18, 2022

Conversation

WillDaSilva
Copy link
Contributor

This PR implements @febuiles' suggestion from #30 (comment):

Maybe it'd make sense to add extra optional parameters to the Action so it accepts head/base revisions. Leaving this open while considering other options (PRs to mitigate this issue in the meantime are welcome!)

Closes #30

@febuiles
Copy link

@WillDaSilva Thanks a ton for this. Was away for a few days and am just catching up with my inbox, will try to go through the PR this week.

Copy link

@Serpentiel Serpentiel left a comment

Choose a reason for hiding this comment

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

lgtm

hfhbd added a commit to hfhbd/bootstrap-compose that referenced this pull request Aug 4, 2022
hfhbd added a commit to hfhbd/bootstrap-compose that referenced this pull request Aug 4, 2022
@febuiles febuiles changed the base branch from main to willdasilva-fork August 18, 2022 13:24
@febuiles febuiles merged commit 54af7c7 into actions:willdasilva-fork Aug 18, 2022
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.

support event types other than pull_request
4 participants