Skip to content

ci(auto-merge): sync workflow#113

Open
caugner wants to merge 1 commit intomainfrom
sync-auto-merge
Open

ci(auto-merge): sync workflow#113
caugner wants to merge 1 commit intomainfrom
sync-auto-merge

Conversation

@caugner
Copy link
Contributor

@caugner caugner commented Mar 18, 2026

Description

Syncs the auto-merge.yml workflow to ensure it uses the reusable workflow from mdn/workflows with the correct configuration:

  • Sets permissions: { contents: read }
  • Uses mdn/workflows/.github/workflows/auto-merge.yml@main
  • Sets auto-merge: true only if the repository's existing workflow already auto-merges
  • Removes the obsolete target-repo input

Related issues and pull requests

Part of mdn/fred#1395.

@caugner caugner requested review from a team and LeoMcA and removed request for a team March 18, 2026 12:56
with:
target-repo: ${{ github.repository }}
secrets:
GH_TOKEN: ${{ secrets.AUTOMERGE_TOKEN }}
Copy link
Contributor Author

Choose a reason for hiding this comment

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

✅ PAT added

uses: mdn/workflows/.github/workflows/auto-merge.yml@main
if: github.repository_owner == 'mdn'
with:
target-repo: ${{ github.repository }}
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Given this repo has required lint/test jobs, can we auto-merge?

Suggested change
target-repo: ${{ github.repository }}
target-repo: ${{ github.repository }}
auto-merge: true

@caugner caugner marked this pull request as ready for review March 18, 2026 13:15
@caugner caugner requested a review from a team as a code owner March 18, 2026 13:15
@caugner caugner marked this pull request as draft March 18, 2026 13:20
@caugner caugner marked this pull request as ready for review March 20, 2026 19:02
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