Skip to content

Remove dcoutts, newhoggy, smelc, and kevinhammond from CODEOWNERS#1177

Merged
Jimbo4350 merged 2 commits into
masterfrom
jordan/update-codeowners
Apr 9, 2026
Merged

Remove dcoutts, newhoggy, smelc, and kevinhammond from CODEOWNERS#1177
Jimbo4350 merged 2 commits into
masterfrom
jordan/update-codeowners

Conversation

@Jimbo4350
Copy link
Copy Markdown
Contributor

@Jimbo4350 Jimbo4350 commented Apr 7, 2026

Summary

Test plan

  • Verify new PRs no longer request reviews from removed users

Changelog

- description: |
    Remove dcoutts, newhoggy, smelc, and kevinhammond from CODEOWNERS
  type:
   - maintenance
  projects:
   - cardano-api

Context

Duncan Coutts, John Ky, Clément Hurlin, and Kevin Hammond are no longer active reviewers on cardano-api. Removing them from CODEOWNERS to stop automatic review requests.

How to trust this PR

Inspect the CODEOWNERS diff — only reviewer names were removed.

Checklist

  • Commit sequence broadly makes sense and commits have useful messages
  • Self-reviewed the diff

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

Updates repository ownership settings to stop automatically requesting reviews from inactive maintainers in the cardano-api area.

Changes:

  • Remove @dcoutts and @newhoggy from the cabal.project ownership entry.
  • Remove @dcoutts and @newhoggy from the general * reviewer group.
  • Remove @newhoggy from README.* and .github ownership entries.

@Jimbo4350 Jimbo4350 force-pushed the jordan/update-codeowners branch from ddcb58c to 545dca6 Compare April 7, 2026 19:10
@Jimbo4350 Jimbo4350 changed the title Remove dcoutts and newhoggy from CODEOWNERS Remove dcoutts, newhoggy, smelc, and kevinhammond from CODEOWNERS Apr 7, 2026
@Jimbo4350 Jimbo4350 force-pushed the jordan/update-codeowners branch from 545dca6 to 5f29100 Compare April 9, 2026 14:15
@carbolymer
Copy link
Copy Markdown
Contributor

I think we should remove John from readme too.

@copilot remove John Ky from README

Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 9, 2026

I think we should remove John from readme too.

@copilot remove John Ky from README

Done — removed John Ky from the "Core maintainers" list in README.md in commit 24070af.

Copilot AI requested a review from carbolymer April 9, 2026 19:08
@Jimbo4350 Jimbo4350 enabled auto-merge April 9, 2026 19:28
@Jimbo4350 Jimbo4350 added this pull request to the merge queue Apr 9, 2026
Merged via the queue into master with commit 62293e4 Apr 9, 2026
33 checks passed
@Jimbo4350 Jimbo4350 deleted the jordan/update-codeowners branch April 9, 2026 20: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.

4 participants