Skip to content

Comments

Tweak logs#485

Merged
rkistner merged 2 commits intomainfrom
tweak-logs
Feb 4, 2026
Merged

Tweak logs#485
rkistner merged 2 commits intomainfrom
tweak-logs

Conversation

@rkistner
Copy link
Contributor

@rkistner rkistner commented Feb 4, 2026

Follow-up to #477.

This tweaks the "Configured logger with level ..." message to use the same logger, so it doesn't appear out of place compared to the rest of the logs.

This also removes the "fatal" log level, which is not a valid one currently.

It is still an issue that the first log lines do not respect the config. I'm not sure if there is a good solution for that when we're loading the config from a file though (although we can technically do the configuration earlier if it's via env vars).

Before:

image

After:

image

@changeset-bot
Copy link

changeset-bot bot commented Feb 4, 2026

⚠️ No Changeset found

Latest commit: 3004718

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@rkistner rkistner merged commit e8ebce7 into main Feb 4, 2026
25 checks passed
@rkistner rkistner deleted the tweak-logs branch February 4, 2026 12:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants