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

feat: Adds --no-progress flag that disables the progress bar #408

Merged
merged 2 commits into from
Dec 22, 2023

Conversation

rznzippy
Copy link
Contributor

@rznzippy rznzippy commented Dec 7, 2023

Closes: #390

Background:
The ora@7 progress bar randomly causes seg faults on M1 Macs.
This change enables a workaround for impacted engineers.

Changes:

  • Adds a support of --no-progress key.
  • Updates help screen copy and the corresponding test.

@crutchcorn
Copy link
Member

Apologies, I see the related issue here now: sindresorhus/ora#229

While I'll upgrade Ora to 8.x, I also see that there's a related issue here as well that denotes removing the spinner anyway.

Reviewing and (hopefully) merging now

@crutchcorn
Copy link
Member

CI/CD failing on Windows but is passing locally. Merging

@crutchcorn crutchcorn merged commit 6f3c9f2 into plopjs:main Dec 22, 2023
2 of 4 checks passed
@rznzippy rznzippy deleted the feat/add-no-progress-option branch December 27, 2023 10:57
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

Successfully merging this pull request may close these issues.

feat: make it possible to disable the progress spinner
2 participants