Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Google Fonts Onboarding: QA FAIL: Copyright notices should match canonical pattern in fonts #6

Closed
eliheuer opened this issue Oct 9, 2023 · 4 comments

Comments

@eliheuer
Copy link

eliheuer commented Oct 9, 2023

Google Fonts onboarding uses FontBakery for QA. All FAIL level checks from the universal, opentype and googlefonts profiles must be passing in order to onboard fonts to Google Fonts. Exceptions can be made if necessary.

See the Quality Assurance section of the Google Fonts Guide for more information: https://googlefonts.github.io/gf-guide/qa.html

🔥 FAIL: Copyright notices match canonical pattern in fonts (com.google.fonts/check/font_copyright)

🔥 FAIL Name Table entry: Copyright notices should match a pattern similar to: "Copyright 2019 The Familyname Project Authors (git url)"
But instead we have got:
"Copyright (c) 2020-21, Reddit, Inc. (https://www.reddit.com/). This Font Software is licensed under the SIL Open Font License, Version 1.1. This license is available with a FAQ at: http://scripts.sil.org/OFL"

See the Google Fonts Guide for more information: https://googlefonts.github.io/gf-guide

@stephenhutchings
Copy link
Collaborator

If this issue is non-critical, Reddit's preference is to leave the copyright as is.

@davelab6
Copy link

davelab6 commented Oct 12, 2023

The copyright in the license file is peculiar in naming the 3 families together; eg the mono has an irrelevant notice about the condensed, the notice will change as the condensed becomes a width axis.

It also doesn't match the notice in the font files, eg the dates are different.

I think if Reddit doesn't want to adopt the "the PROJECT NAME project authors" citation and only use Reddit, Inc, and will set up a CLA so that 3rd party contributions via PR to this repo don't require changing the notice, that's fine with me/Google; fixing the dates to match is trivial.

Giving the url as reddit.com isn't very helpful, IMHO, compared to this repo URL, but again I'm willing to let that go if Reddit insists.

A single notice shared for all 3 families that won't change over time is really my goal.

@stephenhutchings
Copy link
Collaborator

Thanks @davelab6. I noticed the inconsistent years between the licence and fonts, and that should be amended in the most recent 1.012 version. All should now read 2020-2023.

For the other points, I'll pass your comments onto Reddit and get back to you.

@tr000y
Copy link
Contributor

tr000y commented Oct 24, 2023

Hey @davelab6, Reddit Inc has a CLA with Google in place, does that suffice here?

Regarding the url, we're happy to take your suggestion and link out to the repo url instead. cc @stephenhutchings

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

4 participants