Skip to content

MAINT consolidate remote datetime parsing - #2412

Merged
Roman Lutz (romanlutz) merged 2 commits into
microsoft:mainfrom
romanlutz:romanlutz-remote-datetime-parsing
Aug 20, 2026
Merged

MAINT consolidate remote datetime parsing#2412
Roman Lutz (romanlutz) merged 2 commits into
microsoft:mainfrom
romanlutz:romanlutz-remote-datetime-parsing

Conversation

@romanlutz

Copy link
Copy Markdown
Contributor

Description

Remote dataset loaders for 0DIN and PromptIntel duplicated the same ISO 8601 datetime parsing logic. This moves that behavior into _RemoteDatasetLoader so remote loaders share one typed implementation while preserving the existing handling of UTC Z suffixes, explicit offsets, empty values, and invalid input.

Tests and Documentation

  • Added deterministic unit coverage for the timestamp formats returned by both APIs and for empty or invalid input.
  • Ran the focused remote-loader, 0DIN, and PromptIntel unit tests: 92 passed.
  • Ran focused Ruff check and format validation.
  • Documentation was not changed because this internal consolidation does not affect the public API or usage.

Copilot AI added 2 commits August 14, 2026 12:50
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
@hannahwestra25 hannahwestra25 self-assigned this Aug 19, 2026
@romanlutz
Roman Lutz (romanlutz) added this pull request to the merge queue Aug 19, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to no response for status checks Aug 19, 2026
@romanlutz
Roman Lutz (romanlutz) added this pull request to the merge queue Aug 19, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to no response for status checks Aug 19, 2026
@romanlutz
Roman Lutz (romanlutz) added this pull request to the merge queue Aug 20, 2026
Merged via the queue into microsoft:main with commit 37bb737 Aug 20, 2026
54 checks passed
@romanlutz
Roman Lutz (romanlutz) deleted the romanlutz-remote-datetime-parsing branch August 20, 2026 03:57
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.

3 participants