Skip to content

fix: issue with non iso date/datetime values in duck db casting a…#125

Open
georgeRobertson wants to merge 1 commit into
release_v083from
fix/gr-ndit-1663-fix_issue_with_non_iso_timestamps
Open

fix: issue with non iso date/datetime values in duck db casting a…#125
georgeRobertson wants to merge 1 commit into
release_v083from
fix/gr-ndit-1663-fix_issue_with_non_iso_timestamps

Conversation

@georgeRobertson

Copy link
Copy Markdown
Contributor

TLDR of changes

  • fix casting of date/datetime/time values in duckdb after validation in the contract.

What kind of changes does this PR introduce?

Tick all that apply

  • fix: A bug fix. Correlates with PATCH in SemVer
  • feat: A new feature. Correlates with MINOR in SemVer
  • docs: Documentation only changes
  • style: Changes that do not affect the meaning of the code (white-space, formatting, missing semi-colons, etc)
  • refactor: A code change that neither fixes a bug nor adds a feature
  • perf: A code change that improves performance
  • test: Adding missing or correcting existing tests
  • build: Changes that affect the build system or external dependencies (example scopes: pip, docker, npm)
  • ci: Changes to CI configuration files and scripts (example scopes: GitLabCI)

Please check if the PR fulfills these requirements

  • I have read and followed the Contributing guidance
  • Docs have been added / updated
  • Tests and Linting in the CI are passing
  • Changes have been reviewed and approved by a Project Maintainer

@georgeRobertson georgeRobertson requested a review from stevenhsd July 2, 2026 15:09
@sonarqubecloud

sonarqubecloud Bot commented Jul 2, 2026

Copy link
Copy Markdown

@georgeRobertson georgeRobertson changed the title fix: fix issue with non iso date/datetime values in duck db casting a… fix: issue with non iso date/datetime values in duck db casting a… Jul 2, 2026
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