Skip to content

Do not fail for a missing idranges file.#1195

Merged
gouttegd merged 1 commit intomasterfrom
dont-fail-on-missing-id-range-file
Mar 7, 2025
Merged

Do not fail for a missing idranges file.#1195
gouttegd merged 1 commit intomasterfrom
dont-fail-on-missing-id-range-file

Conversation

@gouttegd
Copy link
Contributor

@gouttegd gouttegd commented Mar 5, 2025

We now check the validity of the -idranges.owl file as part of the routine QC checks.

But a project may very well decide that they do not need such a file (if they have their own way of allocating IDs), and in that case they may prefer to remove the file that was generated by the ODK seeding process rather than keeping a useless file around.

The absence of an ID range file should not cause a QC failure. If the file is present, then it must be valid, but its absence can be silently ignored.

We now check the validity of the -idranges.owl file as part of the
routine QC checks.

But a project may very well decide that they do _not_ need such a file
(if they have their own way of allocating IDs), and in that case they
may prefer to remove the file that was generated by the ODK seeding
process rather than keeping a useless file around.

The absence of an ID range file should not cause a QC failure. If the
file is present, then it must be valid, but its absence can be silently
ignored.
@gouttegd gouttegd self-assigned this Mar 5, 2025
@gouttegd gouttegd requested a review from matentzn March 5, 2025 16:19
@gouttegd gouttegd merged commit 7149444 into master Mar 7, 2025
1 check passed
@gouttegd gouttegd deleted the dont-fail-on-missing-id-range-file branch March 7, 2025 15:48
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.

1 participant