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

CLI: --no-no-install-fonts looks ugly replace it with --install-fonts with default to false #322

Closed
CAMOBAP opened this issue Feb 21, 2024 · 11 comments
Assignees

Comments

@CAMOBAP
Copy link
Contributor

CAMOBAP commented Feb 21, 2024

Intro

Recently I faced with issue that metanorma-cli didn't installed required front "Work Sans Medium" metanorma/packed-mn#225

Workaround

call metanorma with --no-no-install-fonts flag

Problem

--no-no-install-fonts looks ugly and better to replace it with install-fonts with default value true as was originally expected

cc @opoudjis @ronaldtse any thoughts

@CAMOBAP CAMOBAP self-assigned this Feb 21, 2024
@opoudjis
Copy link
Contributor

OK by me. We're presumably not disrupting too much usage with this, but retain the original --no-no-install-flags for backward compatibility just in case?

@ronaldtse
Copy link
Contributor

--no-no-install-fonts => ugly... I don't think anyone is using it? Let's just rename it.

@ronaldtse ronaldtse changed the title CLI: --no-no-install-fonts looks ugly replace it with --install-fonts with default to false CLI: --no-no-install-fonts looks ugly replace it with --install-fonts with default to false Feb 22, 2024
opoudjis added a commit that referenced this issue May 28, 2024
…ogress-and-install-fonts-322

feat: remove 'no' prefix in --no-progress and --no-install-fonts CLI flags #322
@ReesePlews
Copy link

this is causing github generate jobs to fail across all repos.

Run cd .
ERROR: "metanorma site generate" was called with arguments [".", "--no-no-install-fonts"]
Usage: "metanorma site generate [SOURCE_PATH]"
Error: Process completed with exit code 1.

please provide a work around for how to fix this on github. thank you.

@ronaldtse
Copy link
Contributor

@CAMOBAP can you help check the GitHub actions?

@CAMOBAP
Copy link
Contributor Author

CAMOBAP commented Jun 5, 2024

@ReesePlews to be on the same page which actions from actions-mn you are using?

@ReesePlews
Copy link

hi @CAMOBAP thanks for looking into this. i have confirmed you have access to the two projects listed below.

one repo is here
https://github.com/metanorma/iso-19135/actions/runs/9377121970/job/25818161033

the other is here
https://github.com/Spatial-Web-Foundation/SWF-Corpus_and_IEEEP2874-D2/actions/runs/9370557418

where are these actions mentioned? are they "per repository" or "something managed by metanorma"? no files to generate the document were edited so i really dont know where they are coming from.

if you need something else, please let me know. thanks

@ronaldtse
Copy link
Contributor

Both repos uses the build-and-publish workflow:

So we have to create an update to the v1 tag for build-and-publish.

@ReesePlews
Copy link

hi @ronaldtse thanks for the info, i looked at the workflows in those repos before adding the issues and i could not find the --no-no-install-fonts as a command parameter. that must be in the "build-and-publish" script. is that something each project controls or are they under metanorma control (on github, or in the instance)?

@ReesePlews
Copy link

solution to fix this issue related to the build-and-publish workflow was discussed here;

i dont quite understand how the changes surrounding the --no-no-install-fonts issue was related to the build-and-publish issue mentioned above. but a manual update to v2 in the generate.yml file in one of the repos solved the problem. the other repo was already using main in the generate.yml file, and seemed to be corrected without any revision the next time the document was generated.

thank you for helping to get this fixed.

@ronaldtse
Copy link
Contributor

This is done, closing.

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

4 participants