[BUG] COLORTERM in combination with FORCE_COLOR does not work anymore

[ThunderKey]

• I've checked docs and closed issues for possible solutions.
• I can't find my issue in the FAQ.

Describe the bug

Commit 1ebf823 broke the combination of "FORCE_COLOR" and "COLORTERM" taken from the environment variables.

I've created a simple test:

import io

from rich.console import Console


def test_force_color():
    console = Console(file=io.StringIO(), _environ={
        "FORCE_COLOR": "1",
        "COLORTERM": "truecolor",
    })
    assert console.is_terminal
    assert console.color_system == "truecolor"

If master or 1ebf823 is checked out it fails, because the color_system is None. If the commit before (b89d036) is checked out the test succeeds.

I guess that the order of when FORCE_COLOR and COLORTERM are interpreted got changed.

Platform

Click to expand

• What platform (Win/Linux/Mac) are you running on? Linux (Manjaro)
• What terminal software are you using? kitty

[github-actions]

Thank you for your issue. Give us a little time to review it.

PS. You might want to check the FAQ if you haven't done so already.

This is an automated reply, generated by FAQtory

[github-actions]

I hope we solved your problem.

If you like using Rich, you might also enjoy Textual

[ThunderKey]

Perfect, thanks! it works like a charm now :)