Skip to content

Conversation

@pixman20
Copy link
Contributor

@pixman20 pixman20 commented Jun 18, 2025

🎟️ Tracking

BRE-848

📔 Objective

Adding permissions to all workflows not being updated by BRE-831
This will prepare for changing the org wide GitHub actions setting to contents:read and packages:read as the default permissions provided to workflows.

📸 Screenshots

⏰ Reminders before review

  • Contributor guidelines followed
  • All formatters and local linters executed and passed
  • Written new unit and / or integration tests where applicable
  • Protected functional changes with optionality (feature flags)
  • Used internationalization (i18n) for all UI strings
  • CI builds passed
  • Communicated to DevOps any deployment requirements
  • Updated any necessary documentation (Confluence, contributing docs) or informed the documentation team

🦮 Reviewer guidelines

  • 👍 (:+1:) or similar for great changes
  • 📝 (:memo:) or ℹ️ (:information_source:) for notes or general info
  • ❓ (:question:) for questions
  • 🤔 (:thinking:) or 💭 (:thought_balloon:) for more open inquiry that's not quite a confirmed issue and could potentially benefit from discussion
  • 🎨 (:art:) for suggestions / improvements
  • ❌ (:x:) or ⚠️ (:warning:) for more significant problems or concerns needing attention
  • 🌱 (:seedling:) or ♻️ (:recycle:) for future improvements or indications of technical debt
  • ⛏ (:pick:) for minor or nitpick changes

@pixman20 pixman20 requested a review from a team as a code owner June 18, 2025 22:02
@pixman20 pixman20 self-assigned this Jun 18, 2025
abergs
abergs previously approved these changes Jun 19, 2025
@pixman20 pixman20 requested a review from abergs June 20, 2025 16:21
@pixman20 pixman20 enabled auto-merge (squash) June 20, 2025 16:22
@pixman20 pixman20 merged commit 7c4e28a into main Jul 14, 2025
5 checks passed
@pixman20 pixman20 deleted the BRE-848-update-workflow-permissions branch July 14, 2025 12:23
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