Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Improve dev first use log messages #489

Merged
merged 5 commits into from
Oct 23, 2024
Merged

Improve dev first use log messages #489

merged 5 commits into from
Oct 23, 2024

Conversation

mattjohnsonpint
Copy link
Collaborator

Description

To improve the first use of launching modus dev I've done the following:

  • In dev, hide the warnings about the DB not being configured. The DB isn't required for most operations, and we'll soon have an alternative.
  • Fix the sort order of messages logged when reading the manifest.

Checklist

  • Code compiles correctly and linting passes locally
  • Entry added to the CHANGELOG.md file describing and linking to this PR

@mattjohnsonpint mattjohnsonpint requested a review from a team as a code owner October 23, 2024 03:11
@mattjohnsonpint mattjohnsonpint merged commit 60520b9 into main Oct 23, 2024
43 checks passed
@mattjohnsonpint mattjohnsonpint deleted the mjp/hyp-2462-hide-database-not-configured-warnings-in-dev branch October 23, 2024 03:28
@mattjohnsonpint mattjohnsonpint mentioned this pull request Oct 23, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kevinmingtarja kevinmingtarja kevinmingtarja approved these changes

Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@mattjohnsonpint @kevinmingtarja