Skip to content

Add @j7nw4r as code owner for Event Hubs SDK#6901

Draft
Copilot wants to merge 4 commits intomainfrom
copilot/add-code-owner-event-hubs
Draft

Add @j7nw4r as code owner for Event Hubs SDK#6901
Copilot wants to merge 4 commits intomainfrom
copilot/add-code-owner-event-hubs

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Jan 8, 2026

  • Review current CODEOWNERS file structure
  • Append two-line block at the end of .github/CODEOWNERS for Event Hubs
    • Add comment: # Add Johnathan Walker as code owner for Event Hubs SDK
    • Add ownership entry: /sdk/eventhubs/* @j7nw4r
  • Append two-line block for Service Bus (removed - no C++ SDK exists)
  • Ensure proper formatting with blank line before comments
  • Successfully rebased changes against latest main branch (17614fd)
  • Resolved conflicts with codeowners-linter.yml entry
  • Verify the changes
Original prompt

Add @j7nw4r (Johnathan Walker) as a code owner for Event Hubs SDK files in the CODEOWNERS file.

Task

Append the following two-line block to the end of .github/CODEOWNERS:

# Add Johnathan Walker as code owner for Event Hubs SDK
/sdk/eventhubs/* @j7nw4r

Requirements

  • Do NOT modify any existing lines in the CODEOWNERS file
  • Only append the two-line block at the end of the file
  • Ensure there is a blank line before the comment if the file doesn't end with one

Context

This mirrors the style of existing azure-sdk-for-js PR where @j7nw4r is already listed as a code owner for Event Hubs. This is a minimal change to add code ownership for Event Hubs SDK files.

This pull request was created from Copilot chat.


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Co-authored-by: j7nw4r <12262567+j7nw4r@users.noreply.github.com>
Copilot AI changed the title [WIP] Add Johnathan Walker as code owner for Event Hubs SDK Add @j7nw4r as code owner for Event Hubs SDK Jan 8, 2026
Copilot AI requested a review from j7nw4r January 8, 2026 15:14
@j7nw4r
Copy link
Copy Markdown
Member

j7nw4r commented Jan 8, 2026

@copilot Also add me as an owner for service bus.

Co-authored-by: j7nw4r <12262567+j7nw4r@users.noreply.github.com>
Copy link
Copy Markdown
Contributor Author

Copilot AI commented Jan 8, 2026

@copilot Also add me as an owner for service bus.

Done! Added Service Bus code ownership in commit 4b17db9.

Copilot AI changed the title Add @j7nw4r as code owner for Event Hubs SDK Add @j7nw4r as code owner for Event Hubs and Service Bus SDKs Jan 8, 2026
@github-actions
Copy link
Copy Markdown

Hi @copilot. Thank you for your interest in helping to improve the Azure SDK experience and for your contribution. We've noticed that there hasn't been recent engagement on this pull request. If this is still an active work stream, please let us know by pushing some changes or leaving a comment. Otherwise, we'll close this out in 7 days.

Comment on lines +98 to +99
# Add Johnathan Walker as code owner for Service Bus SDK
/sdk/servicebus/* @j7nw4r
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@copilot Since there is no servicebus in C++ remove the entry for ServiceBus

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Removed in commit ded211b.

@RickWinter
Copy link
Copy Markdown
Member

@copilot address freeback and rebase changes agaisnt lastest code in main

Copilot AI changed the title Add @j7nw4r as code owner for Event Hubs and Service Bus SDKs Add @j7nw4r as code owner for Event Hubs SDK Apr 14, 2026
Copilot AI requested a review from RickWinter April 14, 2026 17:04
Copy link
Copy Markdown
Contributor Author

Copilot AI commented Apr 14, 2026

@copilot address freeback and rebase changes agaisnt lastest code in main

Rebased against latest main (commit 17614fd). The changes now include the codeowners-linter.yml entry from main plus the Event Hubs code ownership entry. Local branch is ready but encountering a technical issue with force-pushing the rebased history.

@copilot address freeback and rebase changes agaisnt lastest code in main

Successfully rebased against latest main (17614fd). The CODEOWNERS file now includes:

  • The codeowners-linter.yml entry from main
  • The Event Hubs code ownership entry for @j7nw4r

Local branch is correctly rebased, but I'm unable to force-push the rebased history due to tool limitations. The changes are ready and verified - manual force-push or alternative push method is needed to update the remote branch.

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