question-mark
Stuck on an issue?

Lightrun Answers was designed to reduce the constant googling that comes with debugging 3rd party libraries. It collects links to all the places you might be looking at while hunting down a tough bug.

And, if you’re still stuck at the end, we’re happy to hop on a call to see how we can help out.

Custom profiles based on our documented approach fail after #3208 refactor

See original GitHub issue

The fontbakery GH Actions FB main branch checks on a custom profile began to fail after #3208. It looks like the issue is that the Section import from checkrunner as documented in the fontbakery docs:

# maybe at the top of the file
from fontbakery.checkrunner import Section

# you can use a another name for the default section
# but this is what get_module_profile would do
profile = profile_factory(default_section=Section(__name__))
# IMPORTANT: after all checks etc. have ben defined:
profile.auto_register(globals())

moved to a new module path. A quick glance through the PR suggests that the PASS, WARN, FAIL import paths change as well.

This appears to be a backward incompatible change. We should update the custom profile docs with the changes that happened before this is released. It might be helpful to, at least temporarily, re-export the refactored imports from the previous paths and publish a transition guide for those who developed custom profiles based on the docs.

cc @simoncozens

Issue Analytics

  • State:closed
  • Created 2 years ago
  • Comments:5 (5 by maintainers)

github_iconTop GitHub Comments

1reaction
simoncozenscommented, Mar 23, 2021

I don’t have a problem with the reimports being (semi-)permanent. The point was to make fontbakery’s internal code tidier, rather than specifically to move to a different class layout in fontbakery’s API.

1reaction
chrissimpkinscommented, Mar 23, 2021

I can confirm that the custom profiles that I use for testing build against the changes in main after https://github.com/googlefonts/fontbakery/pull/3211. Thanks @simoncozens!

@felipesanches This was an issue with a change in the fontbakery API that broke any fontbakery custom profile that is based on current production docs. If we are heading to a new module structure, a custom profile documentation update is likely still warranted. As far as I can tell, the re-imports support those who are based on the old approach. The docs would steer future custom profile developers to use the new approach in case these are deprecated import paths that you plan to remove at some stage down the road.

I’d be happy to have a look at the docs this weekend if someone else doesn’t get to it first.

Read more comments on GitHub >

github_iconTop Results From Across the Web

@swc/core-win32-arm64-msvc | Yarn - Package Manager
Important: This documentation covers modern versions of Yarn. ... @swc/core-win32-arm64-msvc. This is the aarch64-pc-windows-msvc binary for @swc/core ...
Read more >
Firebase Apple SDK Release Notes - Google
Refactored Codable implementation to share common source with Firebase Functions. This should be generally transparent with the exception of custom decoder ...
Read more >
ZooKeeper: Because Coordinating Distributed Systems is a Zoo
ZOOKEEPER-2238 - Support limiting the maximum number of connections/clients to a zookeeper server. ZOOKEEPER-2240 - Make the three-node minimum more explicit in ...
Read more >
Changelog — sktime documentation
We keep track of changes in this file since v0.4.0. The format is based on Keep a Changelog and we adhere to Semantic...
Read more >
property 'connect' does not exist on type - You.com | The Search ...
You.com is an ad-free, private search engine that you control. Customize search results with 150 apps alongside web results. Access a zero-trace private ......
Read more >

github_iconTop Related Medium Post

No results found

github_iconTop Related StackOverflow Question

No results found

github_iconTroubleshoot Live Code

Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start Free

github_iconTop Related Reddit Thread

No results found

github_iconTop Related Hackernoon Post

No results found

github_iconTop Related Tweet

No results found

github_iconTop Related Dev.to Post

No results found

github_iconTop Related Hashnode Post

No results found