Skip to content

Changes for valid format & required fields in manifest. updated the v… - #24

Merged
Suvinthomas merged 1 commit into
mainfrom
CDSPT-15911-update-manifestvalidator
Nov 10, 2026
Merged

Suvinthomas merged 1 commit into
mainfrom
CDSPT-15911-update-manifestvalidator

Conversation

@Suvinthomas

Copy link
Copy Markdown
Contributor

Version to be released: 0.67

Summary

Removed prepared_by from the list of required manifest properties, making it an optional field.
Retained type validation for prepared_by when it is supplied in the manifest.
Updated the supported format values to align with the revised output format specification.
Added/updated unit tests to verify that manifests validate successfully when prepared_by is omitted.

Changes

Updated REQUIRED_PROPERTIES:

  • Removed prepared_by.

Updated VALID_FORMATS:

  • CSV - Labelled
  • CSV - Unlabelled
  • Text - Labelled
  • Text - Unlabelled
  • Excel
  • Excel - Pivot
  • Parquet - Labelled
  • Parquet - Unlabelled

Updated tests:

Verified prepared_by is no longer treated as a mandatory property.
Existing required property validation remains unchanged for other mandatory fields.
image

Impact

Existing manifests without a prepared_by field will now validate successfully.
Manifests using legacy format names will need to be updated to the new format naming convention.

@scottbw scottbw self-assigned this Nov 10, 2026

@scottbw scottbw left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Reviewed and approved - no changes needed.

@Suvinthomas
Suvinthomas merged commit 395955b into main Nov 10, 2026
0 of 3 checks passed
@Suvinthomas
Suvinthomas deleted the CDSPT-15911-update-manifestvalidator branch November 10, 2026 12:50
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