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

Retrofiting an existing repository #11

Open
aaronbell opened this issue Jun 24, 2021 · 1 comment
Open

Retrofiting an existing repository #11

aaronbell opened this issue Jun 24, 2021 · 1 comment

Comments

@aaronbell
Copy link

aaronbell commented Jun 24, 2021

I'm looking to retrofit existing repositories to the UFR model, and noted that part of the github workflow script creates the fontbakery / proofing documents, then commits them to a separate branch gh-pages.

This requirement seems fine when creating a new repro to house the fonts, or where we are the owner of the repro, but it becomes a hassle when retrofitting a repro owned by someone else because it will require the repro owner to not just merge the PR, but also (a) create a new branch and (b) empty that branch of all content (to keep it clean and follow UFR template model). And if they don't, then every time the repository is updated, they'll receive a "workflow fail" notification. So it is a hassle for the owner either way.

Is there a way that these fontbakery / proofing results / artefacts could be merged in a self-contained way (as in, to the main branch) to make things simple? Or should I just get rid of the automatic workflow process for these sorts of situations? Or are we just expecting owners to create the extra branch?

@vv-monsalve
Copy link
Contributor

Is there a way that these fontbakery / proofing results / artefacts could be merged in a self-contained way (as in, to the main branch) to make things simple?

It could be great, or reported in a new Issue, similarly to what gftools qa allows when using the command gftools qa -f Allison-Regular.ttf --fontbakery --out-url https://github.com/googlefonts/allison

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

No branches or pull requests

2 participants