Skip to content

Update codeowners#2691

Merged
djsmith85 merged 4 commits intomasterfrom
update-codeowner
Aug 15, 2023
Merged

Update codeowners#2691
djsmith85 merged 4 commits intomasterfrom
update-codeowner

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

Updating CODEOWNERS so that translation related files are owned by team-lead-eng -> Needed for the Crowdin Sync PRs
Additionally I have added team-tools as owners of the email-forwarders

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 fedemkr August 15, 2023 10:56
@bitwarden-bot
Copy link

Logo
Checkmarx One – Scan Summary & Detailsf8f7639f-b656-4d2e-a0d4-1437433ec3a9

No New Or Fixed Issues Found

@djsmith85 djsmith85 merged commit 420dc09 into master Aug 15, 2023
@djsmith85 djsmith85 deleted the update-codeowner branch August 15, 2023 13:32
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