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.

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

Migrations #288

Merged
merged 2 commits into from
Nov 25, 2023
Merged

Migrations #288

merged 2 commits into from
Nov 25, 2023

Conversation

antonneubauer
Copy link
Collaborator

@antonneubauer antonneubauer commented Sep 27, 2023

applied migrations, mostly to remove the warnings on server startup.

@ Reviewer: please verify that the red warning labels are not visible anymore for you on server startup

Screenshot 2023-09-27 at 18 45 45

PR checklist

  • tests and linter have passed AFTER local merge
  • at least one other dev reviewed and approved
  • documentation

@github-actions
Copy link

Coverage report

The coverage rate went from 71.43% to 71.43% ⬆️
The branch rate is 65%.

None of the new lines are part of the tested code. Therefore, there is no coverage data about them.

@antonneubauer antonneubauer requested review from BelanaZ and removed request for fynnkroeger September 28, 2023 12:56
Copy link
Collaborator

@BelanaZ BelanaZ left a comment

Choose a reason for hiding this comment

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

Red migrations message is gone 👍

@antonneubauer antonneubauer merged commit ce23960 into main Nov 25, 2023
@antonneubauer antonneubauer deleted the migrations branch November 25, 2023 10:44
henninggaertner pushed a commit that referenced this pull request Nov 26, 2023
* migrated & tracking unused databases

* clarify why db is tracked
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