Skip to content

Commit

Permalink
Deprecation of Shaping profile, and migration of its checks
Browse files Browse the repository at this point in the history
The Shaping profile was also removed and its checks migrated to the Google Fonts profile.
As gfonts was already including those, and no other profile did so, there's no need to add experimental flags to these ones.

(issue fonttools#4801)
  • Loading branch information
felipesanches committed Aug 9, 2024
1 parent 69bc6ff commit 3f64daa
Show file tree
Hide file tree
Showing 7 changed files with 16 additions and 31 deletions.
4 changes: 0 additions & 4 deletions .github/workflows/install_run.yml
Original file line number Diff line number Diff line change
Expand Up @@ -78,10 +78,6 @@ jobs:
name: adobefonts
args: >-
data/test/source-sans-pro/VAR/SourceSansVariable-Roman.ttf
- description: Shaping checks
extras: ".[shaping]"
name: shaping
args: data/test/mada/Mada-Regular.ttf
- description: Google Fonts
extras: ".[googlefonts]"
name: googlefonts
Expand Down
1 change: 1 addition & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,7 @@ A more detailed list of changes is available in the corresponding milestones for
## Upcoming release: 0.12.10 (2024-Jul-??)
### Noteworthy code changes
- The **UFO** profile was removed and its checks were migrated to the **Universal** profile. For that reason, all of them received a temporary "experimental" flag (issue #4801)
- The **shaping** profile was also removed and its checks migrated to the **Google Fonts** profile. As gfonts was already including those, and no other profile did so, there's no need to add experimental flags to these ones. (issue #4801)

### Changes to existing checks
- **EXPERIMENTAL - [com.daltonmaag/check/consistent_curve_type]:**: remove usage of `ufoLib2` APIs. (PR #4802)
Expand Down
1 change: 0 additions & 1 deletion Lib/fontbakery/cli.py
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,6 @@
"microsoft",
"notofonts",
"opentype",
"shaping",
"typenetwork",
"universal",
]
Expand Down
9 changes: 8 additions & 1 deletion Lib/fontbakery/profiles/googlefonts.py
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
PROFILE = {
"include_profiles": ["universal", "outline", "shaping"],
"include_profiles": ["universal", "outline"],
"sections": {
"Article Checks": [
"com.google.fonts/check/article/images",
Expand Down Expand Up @@ -101,6 +101,13 @@
"com.google.fonts/check/repo/vf_has_static_fonts",
"com.google.fonts/check/repo/zip_files",
],
"Shaping Checks": [
"com.google.fonts/check/shaping/regression",
"com.google.fonts/check/shaping/forbidden",
"com.google.fonts/check/shaping/collides",
"com.google.fonts/check/dotted_circle",
"com.google.fonts/check/soft_dotted",
],
"Font File Checks": [
"com.google.fonts/check/aat",
"com.google.fonts/check/canonical_filename",
Expand Down
11 changes: 0 additions & 11 deletions Lib/fontbakery/profiles/shaping.py

This file was deleted.

1 change: 0 additions & 1 deletion docs/source/fontbakery/profiles/index.rst
Original file line number Diff line number Diff line change
Expand Up @@ -15,4 +15,3 @@ profiles
fontval
iso15008
outline
shaping
20 changes: 7 additions & 13 deletions pyproject.toml
Original file line number Diff line number Diff line change
Expand Up @@ -102,6 +102,10 @@ fontwerk = [
]

googlefonts = [
"collidoscope >= 0.5.2",
# 0.5.2 added Python 3.11 wheels
# (see https://github.com/fonttools/fontbakery/issues/3970)

"dehinter >= 3.1.0",
# 3.1.0 added dehinter.font.hint function

Expand All @@ -113,11 +117,13 @@ googlefonts = [
# We cannot use v4 because our protobuf files have been compiled with v3.
# (see https://github.com/fonttools/fontbakery/issues/2200)

"stringbrewer",

"unicodedata2",

"fontbakery[beautifulsoup4]",
"fontbakery[googlefontsalwayslatest]",
"fontbakery[shaping]",
"fontbakery[shaperglot]",
]

iso15008 = []
Expand All @@ -128,17 +134,6 @@ notofonts = [
"fontbakery[googlefonts]",
]

# FIXME: This is not a vendor-specific profile!
shaping = [
"collidoscope >= 0.5.2",
# 0.5.2 added Python 3.11 wheels
# (see https://github.com/fonttools/fontbakery/issues/3970)

"stringbrewer",

"fontbakery[shaperglot]",
]

typenetwork = [
"unicodedata2",

Expand Down Expand Up @@ -167,7 +162,6 @@ all = [
"fontbakery[googlefonts]",
"fontbakery[iso15008]",
"fontbakery[notofonts]",
"fontbakery[shaping]",
"fontbakery[typenetwork]",
]

Expand Down

0 comments on commit 3f64daa

Please sign in to comment.