Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Remove mention of old coverage comment from checklist#685

Merged
Mariatta merged 1 commit into
python:masterpython/devguide:masterfrom
ammaraskar:patch-1ammaraskar/devguide:patch-1Copy head branch name to clipboard
Apr 30, 2021
Merged

Remove mention of old coverage comment from checklist#685
Mariatta merged 1 commit into
python:masterpython/devguide:masterfrom
ammaraskar:patch-1ammaraskar/devguide:patch-1Copy head branch name to clipboard

Conversation

@ammaraskar

Copy link
Copy Markdown
Member

This comment seems to be an old revised mention about codecov's coverage comments from here: https://github.com/python/devguide/pull/80/files#diff-eb1b0e0746ead0f2c05fe91ab3cf886a06b30c20459147869209616b5d2279cbR20-R21

We've long disabled codecov posting comments on pull requests about coverage, and with python/cpython#25679 working to remove the coverage build, we can get rid of this mention.

@Mariatta Mariatta merged commit fb051b7 into python:master Apr 30, 2021
@Mariatta

Copy link
Copy Markdown
Member

Thanks!

AA-Turner pushed a commit to AA-Turner/devguide that referenced this pull request Jun 17, 2022
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

Morty Proxy This is a proxified and sanitized view of the page, visit original site.