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

gha: also test dragonball #2

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

fidencio
Copy link
Member

@fidencio fidencio commented Apr 5, 2023

PR coming from the same repo.

fidencio added 4 commits April 5, 2023 09:49
Let's make this workflow dependent of the commit message check, and only
start it if the commit message check one passes.

This allows us to run this specific workflow using secrets.

Signed-off-by: Fabiano Fidêncio <[email protected]>
The way previously used to get the PR's commit sha can only be used with
`pull_request*` kind of events.

Let's adapt it to the `workflow_run` now that we're using it.

With this change we ended up dropping the PR number from the tarball
suffix, as that's not straightforward to get and, to be honest, not a
unique differentiator that would justify the effort.

Signed-off-by: Fabiano Fidêncio <[email protected]>
As we're using the `workflow_run` event, the checkout action would
pull the **current target branch** instead of the PR one.

Signed-off-by: Fabiano Fidêncio <[email protected]>
FooBar ...

Fixes: #0000

Signed-off-by: Fabiano Fidêncio <[email protected]>
@fidencio fidencio force-pushed the main branch 4 times, most recently from a742113 to 13d857a Compare April 6, 2023 14:56
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.

1 participant