feat(core): Comply with `NO_COLOR` in logs
Summary
Make log coloring dependent on NO_COLOR in prod and dev envs. Already supported on cloud.
Related Linear tickets, Github issues, and Community forum posts
https://linear.app/n8n/issue/PAY-2262
Review / Merge checklist
- [x] PR title and summary are descriptive. (conventions)
- [x] Docs updated or follow-up ticket created.
- ~~Tests included.~~
- ~~PR Labeled with
release/backport(if the PR is an urgent fix that needs to be backported)~~
Codecov Report
Attention: Patch coverage is 0% with 12 lines in your changes missing coverage. Please review.
| Files with missing lines | Patch % | Lines |
|---|---|---|
| packages/core/src/logging/logger.ts | 0.00% | 11 Missing and 1 partial :warning: |
:loudspeaker: Thoughts on this report? Let us know!
n8n
Run #8498
Run Properties:
Passed #8498 •
dfa05c3c8c: 🌳 🖥️ browsers:node18.12.0-chrome107 🤖 ivov 🗃️ e2e/*
| Project |
n8n
|
| Branch Review |
pay-2262-no-color-in-logs
|
| Run status |
|
| Run duration | 04m 48s |
| Commit |
|
| Committer | Iván Ovejero |
| View all properties for this run ↗︎ | |
| Test results | |
|---|---|
|
|
0
|
|
|
3
|
|
|
0
|
|
|
0
|
|
|
483
|
| View all changes introduced in this branch ↗︎ | |
:white_check_mark: All Cypress E2E specs passed
Got released with [email protected]
Got released with [email protected]
Got released with [email protected]
Got released with [email protected]