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

cross port importlib-metadata PR #76 by asottile · Pull Request #13903 · python/cpython · GitHub

/ cpython Public

cross port importlib-metadata PR #76 - #13903

Merged
warsaw merged 1 commit into
python:masterfrom
asottile:importlib_metadata_cross_port_PR_76
Jun 7, 2019
Merged

cross port importlib-metadata PR #76#13903
warsaw merged 1 commit into
python:masterfrom
asottile:importlib_metadata_cross_port_PR_76

Conversation

asottile commented Jun 7, 2019

Copy link
Copy Markdown
Contributor

asottile commented Jun 7, 2019

Copy link
Copy Markdown
Contributor Author

(needs backport to 3.8 only)

warsaw commented Jun 7, 2019

Copy link
Copy Markdown
Member

@Mariatta Interesting buglet: If I add two labels at the same time (i.e. skip-news and skip-issue), it seems like the bot only gets one of them. I had to unapply skip-news and then reapply it.

Mariatta commented Jun 7, 2019

Copy link
Copy Markdown
Member

Hmm this might be a GitHub related issue, a race condition of some sort..

warsaw merged commit 65e5860 into python:master Jun 7, 2019

Copy link
Copy Markdown

@warsaw: Please replace # with GH- in the commit message next time. Thanks!

Copy link
Copy Markdown
Contributor

Thanks @asottile for the PR, and @warsaw for merging it 🌮🎉.. I'm working now to backport this PR to: 3.8.
🐍🍒⛏🤖

warsaw commented Jun 7, 2019

Copy link
Copy Markdown
Member

Hmm this might be a GitHub related issue, a race condition of some sort..

Yeah, probably

Copy link
Copy Markdown

GH-13904 is a backport of this pull request to the 3.8 branch.

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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants


Back | FazBrowse Home | New Git URL