Skip to content

chore: Use new serverpod logging package#5309

Merged
Crazelu merged 8 commits into
serverpod:mainfrom
Crazelu:chore/serverpod-logging
Jun 11, 2026
Merged

chore: Use new serverpod logging package#5309
Crazelu merged 8 commits into
serverpod:mainfrom
Crazelu:chore/serverpod-logging

Conversation

@Crazelu

@Crazelu Crazelu commented Jun 10, 2026

Copy link
Copy Markdown
Collaborator

The logging utilities that were in serverpod_shared have been moved into isolated_object, serverpod_logging and serverpod_logging_cli packages. serverpod_shared now exports serverpod_logging for the least amount of changes downstream. This comes with version bumps for cli_tools, config and serverpod_tui.

Pre-launch Checklist

  • I read the Contribute page and followed the process outlined there for submitting PRs.
  • This update contains only one single feature or bug fix and nothing else. (If you are submitting multiple fixes, please make multiple PRs.)
  • I read and followed the Dart Style Guide and formatted the code with dart format.
  • I listed at least one issue that this PR fixes in the description above.
  • I updated/added relevant documentation (doc comments with ///), and made sure that the documentation follows the same style as other Serverpod documentation. I checked spelling and grammar.
  • I added new tests to check the change I am making.
  • All existing and new tests are passing.
  • Any breaking changes are documented below.

If you need help, consider asking for advice on the discussion board.

Breaking changes

None

@nielsenko

Copy link
Copy Markdown
Collaborator

This is failing due to a bug in cli_tools 0.13.1. We need to fix and roll a new version.

@nielsenko

Copy link
Copy Markdown
Collaborator

This is failing due to a bug in cli_tools 0.13.1. We need to fix and roll a new version.

Fixed in serverpod/cli_tools#125

@Crazelu Crazelu requested a review from a team June 11, 2026 14:40

@marcelomendoncasoares marcelomendoncasoares left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

Now I can go back to the analytics PR, it was awaiting the cli_tools bump :)

@Crazelu Crazelu merged commit d2a861d into serverpod:main Jun 11, 2026
91 checks passed
@github-actions github-actions Bot locked and limited conversation to collaborators Jun 11, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants