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

Re-generate schemas #324

Merged
merged 1 commit into from
Nov 7, 2024
Merged

Re-generate schemas #324

merged 1 commit into from
Nov 7, 2024

Conversation

Milena-Czierlinski
Copy link
Contributor

Readiness checklist

  • I added/updated tests.
  • I ensured that the PR title is good enough for the changelog.
  • I labeled the PR.

The rearranging lets it look more severe in https://github.com/nmshd/runtime/pull/307/files#diff-85c87594e414aa5f153c37403b0dbd816f1a4d9773ba4fbe5cb4e988c6488854 than it actually is.

@Milena-Czierlinski Milena-Czierlinski added the chore Some routine work like updating dependencies label Nov 7, 2024
Copy link

codecov bot commented Nov 7, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Files with missing lines Coverage Δ
packages/runtime/src/useCases/common/Schemas.ts 100.00% <100.00%> (ø)

... and 5 files with indirect coverage changes

@Milena-Czierlinski Milena-Czierlinski marked this pull request as ready for review November 7, 2024 16:46
@Milena-Czierlinski Milena-Czierlinski enabled auto-merge (squash) November 7, 2024 16:47
@Milena-Czierlinski Milena-Czierlinski merged commit 6002115 into main Nov 7, 2024
20 checks passed
@Milena-Czierlinski Milena-Czierlinski deleted the generate-schemas branch November 7, 2024 16:50
@jkoenig134
Copy link
Member

I never want to see such a PR again 😅 WTB a pipeline that prevent from merging when the schema ist noch up2date?

@jkoenig134 jkoenig134 mentioned this pull request Nov 12, 2024
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chore Some routine work like updating dependencies
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants