Skip to content

Update CODEOWNERS for Crowdin PRs#2860

Merged
djsmith85 merged 1 commit intomasterfrom
update-codeowners-for-crowdin
Nov 6, 2023
Merged

Update CODEOWNERS for Crowdin PRs#2860
djsmith85 merged 1 commit intomasterfrom
update-codeowners-for-crowdin

Conversation

@djsmith85
Copy link
Contributor

Type of change

  • Bug fix
  • New feature development
  • Tech debt (refactoring, code cleanup, dependency upgrades, etc)
  • Build/deploy pipeline (DevOps)
  • Other

Objective

Adding all files that get synced with the Crowdin PRs besides en/ into ownership of team-tools-dev

Code changes

  • .github/CODEOWNERS:
    • Set ownership of locales to team-tools-dev
    • Set ownership of locales for the stores to team-tools-dev
    • Remove ownership for en on the stores

Before you submit

  • Please check for formatting errors (dotnet format --verify-no-changes) (required)
  • Please add unit tests where it makes sense to do so (encouraged but not required)
  • If this change requires a documentation update - notify the documentation team
  • If this change has particular deployment requirements - notify the DevOps team

@djsmith85 djsmith85 requested a review from a team November 6, 2023 14:49
@bitwarden-bot
Copy link

Logo
Checkmarx One – Scan Summary & Details8c84b3a6-58c5-45a0-8b99-d05e5601b95a

No New Or Fixed Issues Found

@djsmith85 djsmith85 merged commit 3c94ea4 into master Nov 6, 2023
@djsmith85 djsmith85 deleted the update-codeowners-for-crowdin branch November 6, 2023 15: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.

3 participants