Skip to content

Also search "~/.local/share/fonts/" on Linux#336

Merged
JimBobSquarePants merged 1 commit into
SixLabors:mainfrom
ManuelHu:linux-local-share
Jun 25, 2023
Merged

Also search "~/.local/share/fonts/" on Linux#336
JimBobSquarePants merged 1 commit into
SixLabors:mainfrom
ManuelHu:linux-local-share

Conversation

@ManuelHu
Copy link
Copy Markdown
Contributor

Prerequisites

  • I have written a descriptive pull-request title
  • I have verified that there are no overlapping pull-requests open
  • I have verified that I am following matches the existing coding patterns and practice as demonstrated in the repository. These follow strict Stylecop rules 👮.
  • I have provided test coverage for my change (where applicable)

Description

.local/share/fonts/ is another common location for user provided font files on "modern" Linux platforms, see https://wiki.archlinux.org/title/Fonts#Manual_installation

@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Jun 24, 2023

CLA assistant check
All committers have signed the CLA.

@codecov
Copy link
Copy Markdown

codecov Bot commented Jun 24, 2023

Codecov Report

Merging #336 (ef19313) into main (3d93981) will decrease coverage by 1%.
The diff coverage is 0%.

@@          Coverage Diff          @@
##            main    #336   +/-   ##
=====================================
- Coverage     83%     83%   -1%     
=====================================
  Files        227     227           
  Lines      12858   12859    +1     
  Branches    1838    1838           
=====================================
  Hits       10772   10772           
- Misses      1651    1652    +1     
  Partials     435     435           
Flag Coverage Δ
unittests 83% <0%> (-1%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
src/SixLabors.Fonts/SystemFontCollection.cs 52% <0%> (-1%) ⬇️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

Copy link
Copy Markdown
Member

@JimBobSquarePants JimBobSquarePants left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lovely. Thanks!

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.

3 participants