Skip to content

fix(actions) - expose correct variable name for branch name#194

Merged
localheinz merged 1 commit intoergebnis:mainfrom
jaymecd:patch-1
Jun 15, 2024
Merged

fix(actions) - expose correct variable name for branch name#194
localheinz merged 1 commit intoergebnis:mainfrom
jaymecd:patch-1

Conversation

@jaymecd
Copy link
Contributor

@jaymecd jaymecd commented Apr 30, 2024

This pull request

  • fixes envvar name exposed if workflow was triggered via workflow_run event (github/pull-request/add-label-based-on-branch-name)

@localheinz localheinz self-assigned this Jun 15, 2024
Copy link
Member

@localheinz localheinz left a comment

Choose a reason for hiding this comment

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

👍


if (context.eventName == 'workflow_run') {
core.exportVariable("PULL_REQUEST_NUMBER", context.payload.workflow_run.pull_requests[0].number);
core.exportVariable("PULL_REQUEST_NUMBER", context.payload.workflow_run.pull_requests[0].head.ref);
Copy link
Member

Choose a reason for hiding this comment

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

🤦‍♂️

@localheinz localheinz added the bug label Jun 15, 2024
@localheinz localheinz merged commit 13fefb5 into ergebnis:main Jun 15, 2024
@localheinz
Copy link
Member

Thank you, @jaymecd!

@jaymecd jaymecd deleted the patch-1 branch January 4, 2025 19: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.

2 participants