Skip to content

Don't print the first bracket in values in magenta in error messages#9929

Merged
thufschmitt merged 1 commit intoNixOS:masterfrom
9999years:dont-print-values-in-magenta
Feb 8, 2024
Merged

Don't print the first bracket in values in magenta in error messages#9929
thufschmitt merged 1 commit intoNixOS:masterfrom
9999years:dont-print-values-in-magenta

Conversation

@9999years
Copy link
Contributor

@9999years 9999years commented Feb 4, 2024

Motivation

This fixes the opening bracket of lists/attrsets being printed in magenta, unlike the closing bracket.

Noticed in #9753 (comment)

Before:
image

After:
image

Priorities and Process

Add 👍 to pull requests you find important.

The Nix maintainer team uses a GitHub project board to schedule and track reviews.

@9999years 9999years requested a review from edolstra as a code owner February 4, 2024 04:40
This fixes the opening bracket of lists/attrsets being printed in
magenta, unlike the closing bracket.

NixOS#9753 (comment)
@9999years 9999years force-pushed the dont-print-values-in-magenta branch from e351f5e to 9646d62 Compare February 4, 2024 05:17
@thufschmitt thufschmitt merged commit 46a0625 into NixOS:master Feb 8, 2024
@9999years 9999years deleted the dont-print-values-in-magenta branch February 8, 2024 16:36
tebowy pushed a commit to tebowy/nix that referenced this pull request Jul 11, 2024
…enta

Don't print the first bracket in values in magenta in error messages

(cherry picked from commit 46a0625)
Change-Id: I8435565c87db182116140eaeea9df1243e67ea94
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

2 participants