Skip to content

Conversation

@mabuyo
Copy link

@mabuyo mabuyo commented Dec 8, 2025

I'm not quite sure how to test this without merging 😓

@mabuyo mabuyo requested a review from a team as a code owner December 8, 2025 18:42
@mabuyo mabuyo requested a review from dariuszkuc December 8, 2025 18:42
run: |
gh pr create -B main -H metadata_updates --title 'docs: subgraph library metadata updates' --body 'Created by Github Action'
env:
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
Copy link
Member

Choose a reason for hiding this comment

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

You will definitely need to pass GITHUB_TOKEN with proper permissions. Instead of doing direct merge to main - should we open a PR and auto-merge it if it was opened by a known user and all tests pass?

Copy link
Author

Choose a reason for hiding this comment

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

We'd still need to press the "Enable automerge" on the PR I think? I can't find a way to do it automatically without human intervention.

Copy link
Member

Choose a reason for hiding this comment

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

I think you could use gh pr merge (this enables auto merge on the PR and/or merges it if there are no outstanding pending checks) -> https://cli.github.com/manual/gh_pr_merge

Copy link
Member

Choose a reason for hiding this comment

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

i.e. lets try reverting your changes and add (🤞 token has right permissions)

      - name: Enable auto merge PR
        run: |
          gh pr merge
        env:
            GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}

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.

3 participants