Skip to content

add trailing newline to error messages#1426

Merged
Mic92 merged 2 commits intodirenv:masterfrom
jeremyschlatter:error-message-newline
Jun 15, 2025
Merged

add trailing newline to error messages#1426
Mic92 merged 2 commits intodirenv:masterfrom
jeremyschlatter:error-message-newline

Conversation

@jeremyschlatter
Copy link
Copy Markdown
Contributor

@jeremyschlatter jeremyschlatter commented Apr 25, 2025

Fixes a regression in #1336, which caused error messages to display poorly (or in some cases not at all). See the linked issues.

Fixes #1425
Fixes #1431

Also deletes a duplicate ansi escape code.

errorColor was being used twice in a row.
@Arinerron
Copy link
Copy Markdown

Arinerron commented Jun 11, 2025

PLEASE

Please merge 🙏

@jeremyschlatter
Copy link
Copy Markdown
Contributor Author

@zimbatm @Mic92 any feedback on this PR? From the +1's it looks like a lot of people are impacted by the issue, and the fix is very simple.

@Mic92 Mic92 merged commit 63e9557 into direnv:master Jun 15, 2025
14 checks passed
@Arinerron
Copy link
Copy Markdown

Thank you @Mic92 @jeremyschlatter. Appreciate I didn't have to make the PR, haha!

@jeremyschlatter jeremyschlatter deleted the error-message-newline branch June 16, 2025 17:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

direnv error output gets removed by my terminal Error messages display poorly because they lack a trailing newline

3 participants