Skip to content

Allow all fields in the values section and make output optional - #19

Merged
nvnieuwk merged 3 commits into
mainfrom
spec-changes
Nov 5, 2026
Merged

nvnieuwk merged 3 commits into
mainfrom
spec-changes

Conversation

@nvnieuwk

Copy link
Copy Markdown
Member

This pull request introduces several improvements and changes to the worksheet handling logic, making the output section optional and unifying data field references across worksheet sections. The changes increase flexibility in worksheet configuration, especially for pipelines focused on quality checking, and standardize how data fields are accessed and validated throughout the codebase.

Worksheet Configuration and Data Handling:

  • Made the output section of the worksheet optional, updating related logic and schema validation to reflect this change. This allows for worksheets that do not require an output section, which is particularly useful for quality control pipelines. [1] [2]
  • Unified the way data fields are referenced and validated across input, output, metrics, and values sections by introducing a dataFields set and changing the context object from input to data throughout the codebase. [1] [2] [3] [4] [5] [6] [7] [8]

User-Facing Documentation and Examples:

  • Updated documentation and example worksheet YAMLs to use the new data object instead of input, and to demonstrate the new flexibility in field references. [1] [2]

Testing and Error Messages:

  • Updated test cases and error messages to reference data fields instead of input fields, ensuring consistency and clarity for users.

Changelog:

  • Added a new entry for version 0.5.0, summarizing the key changes for users.

@nvnieuwk
nvnieuwk requested a review from matthdsm October 28, 2026 15:39
@nvnieuwk
nvnieuwk requested a review from jhaezebr October 28, 2026 15:44
@nvnieuwk
nvnieuwk merged commit 32c54fc into main Nov 5, 2026
8 checks passed
@nvnieuwk
nvnieuwk deleted the spec-changes branch November 5, 2026 09:18
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