Skip to content

bpo-39600, IDLE: Remove duplicated font names#18430

Merged
willingc merged 1 commit into
python:masterfrom
vstinner:idle_fonts
Feb 10, 2020
Merged

bpo-39600, IDLE: Remove duplicated font names#18430
willingc merged 1 commit into
python:masterfrom
vstinner:idle_fonts

Conversation

@vstinner

@vstinner vstinner commented Feb 10, 2020

Copy link
Copy Markdown
Member

In the font configuration window, remove duplicated font names.

https://bugs.python.org/issue39600

In the font configuration window, remove duplicated font names.
@vstinner

Copy link
Copy Markdown
Member Author

Before, Cantarell is listed 4 times:

before

After, Cantarell is only shown once:

after

@codecov

codecov Bot commented Feb 10, 2020

Copy link
Copy Markdown

Codecov Report

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

Impacted file tree graph

@@            Coverage Diff            @@
##           master   #18430     +/-   ##
=========================================
  Coverage   82.11%   82.12%             
=========================================
  Files        1955     1954      -1     
  Lines      588949   583604   -5345     
  Branches    44428    44428             
=========================================
- Hits       483624   479258   -4366     
+ Misses      95674    94700    -974     
+ Partials     9651     9646      -5     
Impacted Files Coverage Δ
Lib/distutils/tests/test_bdist_rpm.py 30.00% <0.00%> (-65.00%) ⬇️
Lib/distutils/command/bdist_rpm.py 7.63% <0.00%> (-56.88%) ⬇️
Lib/test/test_urllib2net.py 76.92% <0.00%> (-13.85%) ⬇️
Lib/test/test_smtpnet.py 78.57% <0.00%> (-7.15%) ⬇️
Lib/ftplib.py 63.85% <0.00%> (-6.06%) ⬇️
Lib/test/test_ftplib.py 87.11% <0.00%> (-4.72%) ⬇️
Tools/scripts/db2pickle.py 17.82% <0.00%> (-3.97%) ⬇️
Tools/scripts/pickle2db.py 16.98% <0.00%> (-3.78%) ⬇️
Lib/test/test_socket.py 71.94% <0.00%> (-3.77%) ⬇️
Lib/test/test_asyncio/test_base_events.py 91.84% <0.00%> (-3.30%) ⬇️
... and 321 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 29b3fc0...b908a40. Read the comment docs.

@KenanHArik

Copy link
Copy Markdown

I'm at the PyCascades Sprint, and reviewed this PR. This looks good to me merge in.

@willingc

Copy link
Copy Markdown
Contributor

Thanks @KenanHArik and @hroncok for the reviews. Happy to merge.

@willingc willingc merged commit ed335cf into python:master Feb 10, 2020
@terryjreedy

Copy link
Copy Markdown
Member

Carol, I should have been allowed to review before this was merged.

@miss-islington

Copy link
Copy Markdown
Contributor

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

@miss-islington

Copy link
Copy Markdown
Contributor

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

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Feb 10, 2020
In the font configuration window, remove duplicated font names.
(cherry picked from commit ed335cf)

Co-authored-by: Victor Stinner <vstinner@python.org>
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Feb 10, 2020
In the font configuration window, remove duplicated font names.
(cherry picked from commit ed335cf)

Co-authored-by: Victor Stinner <vstinner@python.org>
@terryjreedy

terryjreedy commented Feb 10, 2020

Copy link
Copy Markdown
Member

@Mariatta, @vstinner
The backport PR numbers
#18437 (3.8) tests ran ('status checks passed') but Azure Pipelines summary not passed.
#18436 (3.7) has a github build error.
bedevere also did not do its issue number, backport, news checks.
I suspect that these all prevent merging. Will re-request.

@miss-islington

Copy link
Copy Markdown
Contributor

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

@miss-islington

Copy link
Copy Markdown
Contributor

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

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Feb 10, 2020
In the font configuration window, remove duplicated font names.
(cherry picked from commit ed335cf)

Co-authored-by: Victor Stinner <vstinner@python.org>
@bedevere-bot

Copy link
Copy Markdown

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

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Feb 10, 2020
In the font configuration window, remove duplicated font names.
(cherry picked from commit ed335cf)

Co-authored-by: Victor Stinner <vstinner@python.org>
@bedevere-bot

Copy link
Copy Markdown

GH-18442 is a backport of this pull request to the 3.7 branch.

@python python deleted a comment from miss-islington Feb 10, 2020
@python python deleted a comment from miss-islington Feb 10, 2020
miss-islington added a commit that referenced this pull request Feb 10, 2020
In the font configuration window, remove duplicated font names.
(cherry picked from commit ed335cf)

Co-authored-by: Victor Stinner <vstinner@python.org>
miss-islington added a commit that referenced this pull request Feb 10, 2020
In the font configuration window, remove duplicated font names.
(cherry picked from commit ed335cf)

Co-authored-by: Victor Stinner <vstinner@python.org>
@vstinner vstinner deleted the idle_fonts branch September 21, 2021 22:49
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.

8 participants