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

[Rerun workflow] update username #1774

Merged
merged 2 commits into from
Jan 7, 2024
Merged

[Rerun workflow] update username #1774

merged 2 commits into from
Jan 7, 2024

Conversation

gurevichdmitry
Copy link
Collaborator

Summary of your changes

The PR_AUTHOR was initially extracted from github.event.workflow_run.head_commit.author.email, and this field may vary based on individual user configurations, leading to potential inconsistencies. To avoid creating specific logic for each case, it is preferable to use github.actor, which is passed through the GitHub context. This PR replaces the usage of the PR_AUTHOR environment variable with github.actor.

Screenshot/Data

Related Issues

Checklist

  • I have added tests that prove my fix is effective or that my feature works
  • I have added the necessary README/documentation (if appropriate)

Introducing a new rule?

@gurevichdmitry gurevichdmitry marked this pull request as ready for review January 4, 2024 16:48
@gurevichdmitry gurevichdmitry requested a review from a team as a code owner January 4, 2024 16:48
Copy link

mergify bot commented Jan 4, 2024

This pull request does not have a backport label. Could you fix it @gurevichdmitry? 🙏
To fixup this pull request, you need to add the backport labels for the needed
branches, such as:

  • backport-v./d./d./d is the label to automatically backport to the 8./d branch. /d is the digit
    NOTE: backport-skip has been added to this pull request.

@mergify mergify bot added the backport-skip label Jan 4, 2024
@gurevichdmitry gurevichdmitry merged commit 1ea37d4 into main Jan 7, 2024
18 checks passed
@gurevichdmitry gurevichdmitry deleted the dg-rerun-user branch January 7, 2024 12:01
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.

2 participants