Skip to content

internal: Set owner for generating tokens for auto-merge#82

Merged
xerial merged 1 commit intomainfrom
xerial-patch-1
May 20, 2025
Merged

internal: Set owner for generating tokens for auto-merge#82
xerial merged 1 commit intomainfrom
xerial-patch-1

Conversation

@xerial
Copy link
Copy Markdown
Member

@xerial xerial commented May 20, 2025

This pull request updates the .github/workflows/auto-merge.yml file to include the repository owner as an input parameter for certain steps in the workflow. This change ensures that the workflow explicitly specifies the repository owner when interacting with GitHub Actions.

Workflow improvements:

  • .github/workflows/auto-merge.yml: Added the owner parameter (${{ github.repository_owner }}) to steps requiring GitHub App authentication to ensure the repository owner is explicitly defined. [1] [2]Description

Related Issue/Task

Checklist

  • This pull request focuses on a single task.
  • The change does not contain security credentials

@amazon-q-developer
Copy link
Copy Markdown
Contributor

⏳ I'm reviewing this pull request for security vulnerabilities and code quality issues. I'll provide an update when I'm done

@github-actions github-actions Bot added the internal Non-user facing changes label May 20, 2025
@amazon-q-developer
Copy link
Copy Markdown
Contributor

✅ I finished the code review, and didn't find any security or code quality issues.

@xerial xerial enabled auto-merge (squash) May 20, 2025 21:27
@xerial xerial merged commit 585bbc4 into main May 20, 2025
10 checks passed
@xerial xerial deleted the xerial-patch-1 branch May 20, 2025 21:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

internal Non-user facing changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant