Skip to content

fix(ci): grant contents:read permission for translation-sync caller#4831

Merged
DeepDiver1975 merged 2 commits intomasterfrom
fix/workflow-permissions
Apr 21, 2026
Merged

fix(ci): grant contents:read permission for translation-sync caller#4831
DeepDiver1975 merged 2 commits intomasterfrom
fix/workflow-permissions

Conversation

@DeepDiver1975
Copy link
Copy Markdown
Member

Summary

  • Aligns caller permissions with the reusable workflow's explicit permissions: contents: read declaration added in owncloud/reusable-workflows@550eb2e
  • Replaces permissions: {} with permissions: contents: read so checkout works on both public and private repos

Test plan

  • Verify the translation-sync workflow run succeeds after merge (next scheduled run or manual trigger)

🤖 Generated with Claude Code

@DeepDiver1975 DeepDiver1975 marked this pull request as ready for review April 20, 2026 13:16
@jesmrec jesmrec force-pushed the fix/workflow-permissions branch from cc00ba5 to ff50033 Compare April 21, 2026 09:30
@DeepDiver1975 DeepDiver1975 merged commit aa4887b into master Apr 21, 2026
12 checks passed
@DeepDiver1975 DeepDiver1975 deleted the fix/workflow-permissions branch April 21, 2026 11:04
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