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

Change how dataset to validate against is specified for filenameExists rule #1491

Merged
merged 29 commits into from
Sep 6, 2024

Conversation

GiaJordan
Copy link
Contributor

@GiaJordan GiaJordan commented Sep 4, 2024

Specify the dataset with files that should be validated against as a parameter in the cli and api instead of in the data model

@GiaJordan GiaJordan marked this pull request as draft September 4, 2024 16:49
@GiaJordan GiaJordan marked this pull request as ready for review September 6, 2024 17:33
Copy link

sonarqubecloud bot commented Sep 6, 2024

@GiaJordan
Copy link
Contributor Author

At the time of writing the tests have passed, comments have been addressed, and this PR can be re-reviewed. If that happens while I'm out then one of the reviewers can merge to develop

Copy link
Collaborator

@BryanFauble BryanFauble left a comment

Choose a reason for hiding this comment

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

LGTM!

@GiaJordan GiaJordan changed the title Develop dataset scope fds 2318 Change how dataset to validate against is specified for filenameExists rule Sep 6, 2024
@GiaJordan GiaJordan merged commit 125787c into develop Sep 6, 2024
7 checks passed
@GiaJordan GiaJordan deleted the develop-dataset-scope-FDS-2318 branch September 6, 2024 23:28
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.

3 participants