Skip to content

remove m-a-ge from maintainers list#2193

Merged
yawpitch merged 1 commit into
masterfrom
cmccandless-patch-1
Mar 21, 2020
Merged

remove m-a-ge from maintainers list#2193
yawpitch merged 1 commit into
masterfrom
cmccandless-patch-1

Conversation

@cmccandless

Copy link
Copy Markdown
Contributor

This maintainer seems to have removed himself from @exercism/python, but not from maintainers.json. Furthermore, I believe his GitHub username has changed, so this is now an invalid entry.

This maintainer seems to have removed himself from @exercism/python, but not from maintainers.json. Furthermore, I believe his GitHub username has changed, so this is now an invalid entry.
@cmccandless cmccandless requested a review from a team as a code owner March 20, 2020 13:07

@yawpitch yawpitch left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I'd consider moving them to alumnus: true, but there's no actual detail there and if the username has changed it'd effectively be an entirely new entry for them to re-up, so this LGTM.

@yawpitch yawpitch merged commit f5a094e into master Mar 21, 2020
@yawpitch yawpitch deleted the cmccandless-patch-1 branch March 21, 2020 10:08
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.

2 participants