FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Add barneygale to CODEOWNERS for pathlib by barneygale · Pull Request #100808 · python/cpython · GitHub

/ cpython Public

Add barneygale to CODEOWNERS for pathlib - #100808

Merged
AlexWaygood merged 1 commit into
python:mainfrom
barneygale:codeowners-barneygale
Jan 6, 2023
ConversationCommits (1)ChecksFiles changed
Merged

Add barneygale to CODEOWNERS for pathlib#100808
AlexWaygood merged 1 commit into
python:mainfrom
barneygale:codeowners-barneygale

Conversation

barneygale commented Jan 6, 2023 •
edited by miss-islington
Loading

Copy link
Copy Markdown
Contributor

As @AlexWaygood suggested!

Automerge-Triggered-By: GH:AlexWaygood

Automerge-Triggered-By: GH:AlexWaygood

barneygale requested a review from AlexWaygood January 6, 2023 19:46

AlexWaygood commented Jan 6, 2023 •
edited
Loading

Copy link
Copy Markdown
Member

Oh, you could probably add yourself for some os.path/ntpath/posixpath globs as well, if you want to? (Depends on how much stuff you want to be pinged for :)

barneygale commented Jan 6, 2023

Copy link
Copy Markdown
Contributor Author

Hmm, maybe don't worry for now, but I might open another PR in a few weeks or months?

AlexWaygood commented Jan 6, 2023

Copy link
Copy Markdown
Member

Sure!

miss-islington commented Jan 6, 2023

Copy link
Copy Markdown
Contributor

Status check is done, and it's a success ✅.

AlexWaygood merged commit 26ff436 into python:main Jan 6, 2023

AlexWaygood commented Jan 6, 2023

Copy link
Copy Markdown
Member

We could also backport this to 3.11 and 3.10 if you wanna get pinged on backports as well @barneygale? (Up to you, though I think these are normally backported.)

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Show hidden characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

ezio-melotti ezio-melotti approved these changes
AlexWaygood AlexWaygood approved these changes

Assignees

No one assigned

Labels

skip issue skip news

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

Back | FazBrowse Home | New Git URL