Skip to content

Add CODEOWNERS entry for platform team#15609

Merged
gre-ledger merged 1 commit intodevelopfrom
support/codeowners-platform
Mar 20, 2026
Merged

Add CODEOWNERS entry for platform team#15609
gre-ledger merged 1 commit intodevelopfrom
support/codeowners-platform

Conversation

@gre-ledger
Copy link
Contributor

CODEOWNERS to be owned by platform team

Copilot AI review requested due to automatic review settings March 20, 2026 11:45
@github-actions
Copy link
Contributor

Web Tools Build Status

Build Status Deployment
Web Tools Build ⏭️ Skipped
Native Storybook Build ⏭️ Skipped
React Storybook Build ⏭️ Skipped

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates repository review ownership so the platform team is requested for changes affecting the root CODEOWNERS configuration.

Changes:

  • Adds an ownership rule assigning the root CODEOWNERS file to @ledgerhq/platform.

apps/ledger-live-mobile/fastlane/Fastfile @ledgerhq/wallet-ci

# Platform & Wallet XP teams catchall
CODEOWNERS @ledgerhq/platform
Copy link

Copilot AI Mar 20, 2026

Choose a reason for hiding this comment

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

The pattern CODEOWNERS is not anchored and will match any file named CODEOWNERS anywhere in the repo (e.g. .github/CODEOWNERS), which could unintentionally assign ownership beyond the root file. Prefer an anchored pattern like /CODEOWNERS to target only the repository root CODEOWNERS file.

Suggested change
CODEOWNERS @ledgerhq/platform
/CODEOWNERS @ledgerhq/platform

Copilot uses AI. Check for mistakes.
@gre-ledger gre-ledger merged commit d270b6f into develop Mar 20, 2026
57 of 58 checks passed
@gre-ledger gre-ledger deleted the support/codeowners-platform branch March 20, 2026 16:20
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