Skip to content

fix: respect NO_COLOR env variable in dev format (#302) - #366

Closed
marceli1404 wants to merge 1 commit into
expressjs:masterfrom
marceli1404:fix/no-color-support
Closed

fix: respect NO_COLOR env variable in dev format (#302)#366
marceli1404 wants to merge 1 commit into
expressjs:masterfrom
marceli1404:fix/no-color-support

Conversation

@marceli1404

Copy link
Copy Markdown

Description

Check \process.env.NO_COLOR\ and skip ANSI color codes when set, per the NO_COLOR standard (https://no-color.org/).

Closes #302

@marceli1404

Copy link
Copy Markdown
Author

Friendly bump on this one — morgan('dev') now respects the NO_COLOR standard (#302). The dev format skips ANSI codes when NO_COLOR is set. All 88 tests pass.

@krzysdz

krzysdz commented Aug 20, 2026

Copy link
Copy Markdown

Closing in favour of the first PR - #303. If you have any thoughts about it or possibly better ideas, please join the discussion there.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

morgan('dev') does not respect NO_COLOR=1 environment variable

2 participants