Skip to content

fix: remove unexpected value from actions workflow yml#96

Merged
satsukies merged 1 commit into
mainfrom
fix/remove-unexpected-value
Jul 23, 2026
Merged

fix: remove unexpected value from actions workflow yml#96
satsukies merged 1 commit into
mainfrom
fix/remove-unexpected-value

Conversation

@satsukies

Copy link
Copy Markdown
Member

close #95

We want to permit to auto-merge, but misunderstood it need the 'workflow' scope permission.
GitHub Actions workflow doesn't have a syntax of 'workflows'.
ref. https://docs.github.com/ja/actions/reference/workflows-and-actions/workflow-syntax#permissions

We already have enough permissions for auto-merge, so I removed it.

BTW: That is provided for permit scope for PAT/OAuth token settings.
ref. https://docs.github.com/ja/apps/oauth-apps/building-oauth-apps/scopes-for-oauth-apps#available-scopes

@satsukies
satsukies requested a review from enomoto-kazuya July 23, 2026 05:08
@satsukies satsukies self-assigned this Jul 23, 2026
@satsukies satsukies linked an issue Jul 23, 2026 that may be closed by this pull request
@github-actions

Copy link
Copy Markdown
Contributor

DeployGate Upload Information

Item Content
🔄 Revision 120
📱 App Details View on DeployGate
🔗 Distribution Page https://deploygate.com/distributions/4e531da5dfd007964eb0340bba094e8fbfbe89f2
📲 Open on Mobile QR Code

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Fixes the Dependabot auto-merge workflow failing due to an invalid permissions key (workflows) in the GitHub Actions workflow YAML, aligning the workflow with supported GitHub Actions permission syntax.

Changes:

  • Removed the unsupported permissions: workflows: write entry from the Dependabot auto-merge workflow.
  • Kept existing required permissions (contents: write, pull-requests: write) intact for auto-merge behavior.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@enomoto-kazuya enomoto-kazuya left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@satsukies

Copy link
Copy Markdown
Member Author

Thank you!

@satsukies
satsukies merged commit 361601a into main Jul 23, 2026
6 checks passed
@satsukies
satsukies deleted the fix/remove-unexpected-value branch July 23, 2026 06:03
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.

'Dependabot auto-merge' Actions always failed

3 participants