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

Add verification of initialization data in the database #21

Merged
merged 2 commits into from
Apr 30, 2024
Merged

Conversation

raczu
Copy link
Collaborator

@raczu raczu commented Apr 30, 2024

Description

Added a workflow that, which at each pull request verifies the correctness of sql files used to initalize the database and development data. Additionally, it fixes wrong path in artifact upload for the unit tests workflow.

Motivation

Workflow was added to avoid potential issues resulting from the mistakes/errors occurring in sql files.

Related issue and/or ticket

RSN-50
This PR closes #20

@raczu raczu added scope: backend Indicates that PR relates to backend changes. scope: ci/cd Indicates that PR relates to workflows changes. type: feature request Issues categorised as feature request. labels Apr 30, 2024
@raczu raczu requested a review from wzarek April 30, 2024 01:56
@raczu
Copy link
Collaborator Author

raczu commented Apr 30, 2024

The new workflow is not passing due to existing error described in #16.

@wzarek wzarek added the status: ready for review Determines that PR is ready for review. label Apr 30, 2024
Copy link
Owner

@wzarek wzarek left a comment

Choose a reason for hiding this comment

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

OK

@wzarek wzarek added type: feature PR's categorised as feature. and removed type: feature request Issues categorised as feature request. labels Apr 30, 2024
@raczu raczu merged commit f4a59b9 into main Apr 30, 2024
6 of 7 checks passed
@wzarek wzarek deleted the RSN-50 branch May 8, 2024 16:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
scope: backend Indicates that PR relates to backend changes. scope: ci/cd Indicates that PR relates to workflows changes. status: ready for review Determines that PR is ready for review. type: feature PR's categorised as feature.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Verify proper initialization of data in the database
2 participants