Skip to content

Conversation

@matejchalk
Copy link
Collaborator

The immediate motivation for this change is to avoid --no-progress argument being used in CI, which is part of an effort to avoid extra CLI arguments which restrict cache reuse in Nx monorepos.

In the future, we'd like to get rid of the progress bar completely and replace it with a spinner. Either way, it would make sense to only show these UI elements when not in running in CI. The --progress argument may be removed in the future though, we'll see. The future improvements are part of a larger effort, see:

I've created some general purpose utility functions for correctly parsing boolean environment variables, so it now supports conventions like true/false, 1/0, yes/no, on/off, etc. I've reused those for the CP_VERBOSE variable as well.

@github-actions github-actions bot added 📖 Project documentation improvements or additions to the project documentation 🔬 testing writing tests 🧩 cli 🧩 utils 🧩 ci labels Jul 29, 2025
@matejchalk matejchalk merged commit b0ed2fa into main Jul 30, 2025
15 of 17 checks passed
@matejchalk matejchalk deleted the progress-off-in-ci branch July 30, 2025 15:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🧩 ci 🧩 cli 📖 Project documentation improvements or additions to the project documentation 🔬 testing writing tests 🧩 utils

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants