Skip to content

Allow SasData, and SasMeasurement to be instantiated without metadata. - #247

Merged
rprospero merged 4 commits into
refactor_24from
refactor_24_empty_metadata
Oct 19, 2026
Merged

rprospero merged 4 commits into
refactor_24from
refactor_24_empty_metadata

Conversation

@jamescrake-merani

Copy link
Copy Markdown
Contributor

If no metadata is provided to the constructor, then the metadata will just get initialised as all empty.

jamescrake-merani and others added 4 commits October 9, 2026 10:40
SasData, and SasMeasurement no longer require metadata to be provided
for instantiation. If none is provided, they will just init empty metadata.
Tests were failing because of this.
codescene-access[bot]

This comment was marked as outdated.

@codescene-access codescene-access Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

No quality gates enabled for this code.

See analysis details in CodeScene

Quality Gate Profile: Custom Configuration
Install CodeScene MCP: safeguard and uplift AI-generated code. Catch issues early with our IDE extension and CLI tool.

@rprospero rprospero left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This will definitely help with the ASCII reader

@rprospero
rprospero merged commit ce48485 into refactor_24 Oct 19, 2026
17 checks passed
@rprospero
rprospero deleted the refactor_24_empty_metadata branch October 19, 2026 11:10
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