Skip to content

Simplify auto merge command in action.yml#20

Merged
shunkakinoki merged 2 commits intomainfrom
shunkakinoki-patch-1
Sep 25, 2025
Merged

Simplify auto merge command in action.yml#20
shunkakinoki merged 2 commits intomainfrom
shunkakinoki-patch-1

Conversation

@shunkakinoki
Copy link
Copy Markdown
Owner

Removed merge method input from auto merge command.

Removed merge method input from auto merge command.
Copilot AI review requested due to automatic review settings September 25, 2025 04:15
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR simplifies the auto merge command by removing the merge method input parameter and updating the GitHub token environment variable name.

  • Removes merge method parameter from the gh pr merge command, defaulting to GitHub's default merge behavior
  • Updates environment variable from GITHUB_TOKEN to GH_TOKEN for consistency with GitHub CLI best practices

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

shell: bash
run: |
gh pr merge ${{ github.event.pull_request.number }} --auto --${{ inputs.merge-method }}
gh pr merge ${{ github.event.pull_request.number }} --auto
Copy link

Copilot AI Sep 25, 2025

Choose a reason for hiding this comment

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

Removing the merge method parameter eliminates the ability to specify merge, squash, or rebase strategies. This breaking change reduces flexibility for consumers who may need different merge strategies for different repositories or workflows.

Copilot uses AI. Check for mistakes.
@shunkakinoki shunkakinoki enabled auto-merge (squash) September 25, 2025 04:16
@shunkakinoki shunkakinoki merged commit 6702534 into main Sep 25, 2025
11 checks passed
@shunkakinoki shunkakinoki deleted the shunkakinoki-patch-1 branch September 25, 2025 04:17
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.

2 participants