Skip to content

Removed inactive accounts from CODEOWNERS#28969

Merged
wyunchi-ms merged 1 commit intomainfrom
isra-fel-patch-1
Dec 12, 2025
Merged

Removed inactive accounts from CODEOWNERS#28969
wyunchi-ms merged 1 commit intomainfrom
isra-fel-patch-1

Conversation

@isra-fel
Copy link
Copy Markdown
Member

Description

Removed accounts that no longer have access to the repo.

CODEOWNERS errors
Unknown owner on line 4: make sure @pilor exists and has write access to the repository
/src/PolicyInsights/  @pilor @wyunchi-ms @dolauli @isra-fel @VeryEarly @YanaXu @vidai-msft @NoriZC @notyashhh @Pan-Qi @DanielMicrosoft
Unknown owner on line 5: make sure @wiboris exists and has write access to the repository
/src/Batch/ @wiboris @dpwatrous @wyunchi-ms @dolauli @isra-fel @VeryEarly @YanaXu @vidai-msft @NoriZC @notyashhh @Pan-Qi @DanielMicrosoft
Unknown owner on line 5: make sure @dpwatrous exists and has write access to the repository
/src/Batch/ @wiboris @dpwatrous @wyunchi-ms @dolauli @isra-fel @VeryEarly @YanaXu @vidai-msft @NoriZC @notyashhh @Pan-Qi @DanielMicrosoft
Unknown owner on line 9: make sure @marcodalessandro exists and has write access to the repository
/src/DeviceRegistry/ @marcodalessandro @rohankhandelwal @riteshrao @davidemontanari @wyunchi-ms @dolauli @isra-fel @VeryEarly @YanaXu @vidai-msft @NoriZC @notyashhh @Pan-Qi @DanielMicrosoft
Unknown owner on line 9: make sure @rohankhandelwal exists and has write access to the repository
…ceRegistry/ @marcodalessandro @rohankhandelwal @riteshrao @davidemontanari @wyunchi-ms @dolauli @isra-fel @VeryEarly @YanaXu @vidai-msft @NoriZC @notyashhh @Pan-Qi @DanielMicrosoft
Unknown owner on line 9: make sure @riteshrao exists and has write access to the repository
…odalessandro @rohankhandelwal @riteshrao @davidemontanari @wyunchi-ms @dolauli @isra-fel @VeryEarly @YanaXu @vidai-msft @NoriZC @notyashhh @Pan-Qi @DanielMicrosoft
Unknown owner on line 9: make sure @davidemontanari exists and has write access to the repository
…o @rohankhandelwal @riteshrao @davidemontanari @wyunchi-ms @dolauli @isra-fel @VeryEarly @YanaXu @vidai-msft @NoriZC @notyashhh @Pan-Qi @DanielMicrosoft

Mandatory Checklist

  • SHOULD update ChangeLog.md file(s) appropriately
    • Update src/{{SERVICE}}/{{SERVICE}}/ChangeLog.md.
      • A snippet outlining the change(s) made in the PR should be written under the ## Upcoming Release header in the past tense.
    • Should not change ChangeLog.md if no new release is required, such as fixing test case only.
  • SHOULD regenerate markdown help files if there is cmdlet API change. Instruction
  • SHOULD have proper test coverage for changes in pull request.
  • SHOULD NOT adjust version of module manually in pull request

Copilot AI review requested due to automatic review settings December 11, 2025 07:29
@azure-client-tools-bot-prd
Copy link
Copy Markdown

Thanks for your contribution! The pull request validation has started. Please revisit this comment for updated status.

Copy link
Copy Markdown
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

This PR removes inactive GitHub accounts from the CODEOWNERS file to resolve validation errors. The changes clean up references to accounts that no longer have write access to the repository, ensuring the CODEOWNERS file functions correctly.

Key Changes

  • Removed inactive account @pilor from PolicyInsights module ownership
  • Removed inactive accounts @wiboris and @dpwatrous from Batch module ownership
  • Removed inactive accounts @marcodalessandro, @rohankhandelwal, @riteshrao, and @davidemontanari from DeviceRegistry module ownership, along with the entire DeviceRegistry CODEOWNERS entry

# Make sure Device Registry team gets notified of any DeviceRegistry PRs:
# PRLabel: %Device Registry
/src/DeviceRegistry/ @marcodalessandro @rohankhandelwal @riteshrao @davidemontanari @wyunchi-ms @dolauli @isra-fel @VeryEarly @YanaXu @vidai-msft @NoriZC @notyashhh @Pan-Qi @DanielMicrosoft

Copy link

Copilot AI Dec 11, 2025

Choose a reason for hiding this comment

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

The removal of CODEOWNERS entries for PolicyInsights, Batch, and DeviceRegistry modules means these modules will no longer have dedicated code owners. While the inactive accounts needed to be removed, consider whether these modules should have alternative code owners assigned, or if they should rely on the default Azure PowerShell team members only. This could impact code review workflow and accountability for these modules.

Suggested change
# Assign code owners for PolicyInsights, Batch, and DeviceRegistry modules to ensure accountability
/src/PolicyInsights/ @wyunchi-ms @dolauli @isra-fel @VeryEarly @YanaXu @vidai-msft @NoriZC @notyashhh @Pan-Qi @DanielMicrosoft
/src/Batch/ @wyunchi-ms @dolauli @isra-fel @VeryEarly @YanaXu @vidai-msft @NoriZC @notyashhh @Pan-Qi @DanielMicrosoft
/src/DeviceRegistry/ @wyunchi-ms @dolauli @isra-fel @VeryEarly @YanaXu @vidai-msft @NoriZC @notyashhh @Pan-Qi @DanielMicrosoft

Copilot uses AI. Check for mistakes.
@wyunchi-ms wyunchi-ms merged commit 0d212cd into main Dec 12, 2025
18 checks passed
@wyunchi-ms wyunchi-ms deleted the isra-fel-patch-1 branch December 12, 2025 03:41
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