Skip to content

Respect FORCE_COLOR environment variable#10315

Merged
charliermarsh merged 1 commit intomainfrom
charlie/ter
Jan 6, 2025
Merged

Respect FORCE_COLOR environment variable#10315
charliermarsh merged 1 commit intomainfrom
charlie/ter

Conversation

@charliermarsh
Copy link
Member

Summary

Closes #10303.

global = true,
long,
value_enum,
default_value = "auto",
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The presence of the default meant we always used Auto :(

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Does it need to be removed from

#[arg(long, default_value = "auto", conflicts_with = "no_color", hide = true)]
as well?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think it doesn't matter in practice, but yeah it makes sense to keep them in-sync. Thanks!

@charliermarsh charliermarsh added bug Something isn't working cli Related to the command line interface labels Jan 6, 2025

/// Control colors in output.
///
/// By default, uv will use colors when writing to a terminal.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
/// By default, uv will use colors when writing to a terminal.
/// By default, uv will use automatically detect support for colors when writing to a terminal.

@charliermarsh charliermarsh merged commit f7fc85f into main Jan 6, 2025
55 checks passed
@charliermarsh charliermarsh deleted the charlie/ter branch January 6, 2025 02:18
tmeijn pushed a commit to tmeijn/dotfiles that referenced this pull request Jan 8, 2025
This MR contains the following updates:

| Package | Update | Change |
|---|---|---|
| [astral-sh/uv](https://github.com/astral-sh/uv) | patch | `0.5.14` -> `0.5.15` |

MR created with the help of [el-capitano/tools/renovate-bot](https://gitlab.com/el-capitano/tools/renovate-bot).

**Proposed changes to behavior should be submitted there as MRs.**

---

### Release Notes

<details>
<summary>astral-sh/uv (astral-sh/uv)</summary>

### [`v0.5.15`](https://github.com/astral-sh/uv/blob/HEAD/CHANGELOG.md#0515)

[Compare Source](astral-sh/uv@0.5.14...0.5.15)

##### Python

The managed Python distributions have been updated, including:

-   Python 3.14.0a3 support on macOS and Linux
-   Performance improvements
-   Fixes to SQLite feature detection

See the [`python-build-standalone` release notes](https://github.com/astral-sh/python-build-standalone/releases/tag/20250106) for more details.

##### Enhancements

-   Respect `FORCE_COLOR` environment variable ([#&#8203;10315](astral-sh/uv#10315))

##### Performance

-   Avoid generating unused hashes during `uv lock` ([#&#8203;10307](astral-sh/uv#10307))
-   Visit source distributions before wheels ([#&#8203;10291](astral-sh/uv#10291))

##### Bug fixes

-   Avoid downgrading packages when `--upgrade` is provided ([#&#8203;10097](astral-sh/uv#10097))
-   Extract supported architectures from wheel tags ([#&#8203;10179](astral-sh/uv#10179))
-   Redact new index credentials in `uv add` ([#&#8203;10329](astral-sh/uv#10329))

##### Documentation

-   Clarify `exclude-newer` only allows full timestamps in settings documentation ([#&#8203;9135](astral-sh/uv#9135))
-   Tweak script `--no-project` comment ([#&#8203;10331](astral-sh/uv#10331))
-   Update copyright year ([#&#8203;10297](astral-sh/uv#10297))
-   Add instructinos for installing with Scoop ([#&#8203;10332](astral-sh/uv#10332))

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied.

♻ **Rebasing**: Whenever MR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 **Ignore**: Close this MR and you won't be reminded about this update again.

---

 - [ ] <!-- rebase-check -->If you want to rebase/retry this MR, check this box

---

This MR has been generated by [Renovate Bot](https://github.com/renovatebot/renovate).
<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzOS45MS4yIiwidXBkYXRlZEluVmVyIjoiMzkuOTEuMiIsInRhcmdldEJyYW5jaCI6Im1haW4iLCJsYWJlbHMiOlsiUmVub3ZhdGUgQm90Il19-->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working cli Related to the command line interface

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Setting FORCE_COLOR=true does not seem to work (in GH actions at least)

2 participants