Skip to content

chore: convert update readme to use automerged pull request#98

Merged
abannachGrafana merged 1 commit into
mainfrom
adam.bannach/make-readme-update-auto-pr
Apr 21, 2025
Merged

chore: convert update readme to use automerged pull request#98
abannachGrafana merged 1 commit into
mainfrom
adam.bannach/make-readme-update-auto-pr

Conversation

@abannachGrafana
Copy link
Copy Markdown
Contributor

No description provided.

@abannachGrafana abannachGrafana requested a review from Copilot April 21, 2025 21:19
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 automates the process of updating the README by converting the update into an auto-merged pull request.

  • Increased permissions (switching contents from read-only to write and adding pull-requests write permission) to enable automated operations.
  • Modified the workflow to create a new branch, commit the README changes, and auto-create, review, and merge the PR.

run: gh pr review --approve "$PR_URL" && gh pr merge --auto --squash "$PR_URL"
env:
PR_URL: ${{steps.cpr.outputs.PR_URL}}
GH_TOKEN: ${{ github.token }}
Copy link

Copilot AI Apr 21, 2025

Choose a reason for hiding this comment

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

The auto-merge step uses a different token (${{ github.token }}) than earlier steps that rely on the app-token. Consider using the same token throughout to avoid potential permission inconsistencies.

Suggested change
GH_TOKEN: ${{ github.token }}
GH_TOKEN: ${{ steps.app-token.outputs.token }}

Copilot uses AI. Check for mistakes.
@abannachGrafana abannachGrafana merged commit 66c9dc3 into main Apr 21, 2025
16 checks passed
@abannachGrafana abannachGrafana deleted the adam.bannach/make-readme-update-auto-pr branch April 21, 2025 21:20
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