Skip to content

bpo-39573: Update clinic to use Py_IS_TYPE macro - #18507

Merged
vstinner merged 2 commits into
python:masterfrom
corona10:bpo-39573-clinic
Feb 14, 2020
Merged

bpo-39573: Update clinic to use Py_IS_TYPE macro#18507
vstinner merged 2 commits into
python:masterfrom
corona10:bpo-39573-clinic

Conversation

@corona10

@corona10corona10 commented Feb 14, 2020

Copy link
Copy Markdown
Member

@codecov

codecovBot commented Feb 14, 2020

Copy link
Copy Markdown

Codecov Report

Merging #18507 into master will increase coverage by 2.65%.
The diff coverage is n/a.

Impacted file tree graph

@@ Coverage Diff @@## master #18507 +/- ##
===========================================
+ Coverage 79.46% 82.12% +2.65% 
===========================================
Files 384 1955 +1571 Lines 169290 583979 +414689 Branches 0 44450 +44450 ===========================================
+ Hits 134529 479572 +345043 - Misses 34761 94765 +60004 - Partials 0 9642 +9642 
Impacted FilesCoverage Δ
Programs/_testembed.c89.43% <0.00%> (-0.13%)⬇️
Lib/test/test_reprlib.py95.75% <0.00%> (ø)
Lib/modulefinder.py67.44% <0.00%> (ø)
Lib/distutils/tests/support.py85.00% <0.00%> (ø)
Lib/email/mime/base.py100.00% <0.00%> (ø)
Lib/tkinter/test/test_ttk/test_widgets.py97.43% <0.00%> (ø)
Lib/importlib/abc.py93.44% <0.00%> (ø)
Lib/test/test_property.py90.32% <0.00%> (ø)
Lib/test/test_importlib/test_open.py95.23% <0.00%> (ø)
Lib/unittest/test/testmock/support.py87.50% <0.00%> (ø)
... and 1564 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f632736...79b5e0b. Read the comment docs.

@vstinner
vstinner merged commit 9aeb0ef into python:masterFeb 14, 2020
@vstinner

Copy link
Copy Markdown
Member

Merged, thanks. I just replace "macro" with "function" in the commit message :-)

Sign up for freeto 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

@corona10@vstinner@the-knights-who-say-ni@bedevere-bot