Skip to content

Make datetime.time json serializable, to string ISO format.#701

Merged
bgunebakan merged 7 commits into
mainfrom
fix/datetime-time-serializable
Jun 2, 2026
Merged

Make datetime.time json serializable, to string ISO format.#701
bgunebakan merged 7 commits into
mainfrom
fix/datetime-time-serializable

Conversation

@surister
Copy link
Copy Markdown
Contributor

Summary of the changes / Why this is an improvement

Fixes #696

Checklist

  • Link to issue this PR refers to (if applicable): Fixes #???

@surister surister force-pushed the fix/datetime-time-serializable branch from c77c110 to a4fbe65 Compare February 25, 2025 08:22
@surister surister force-pushed the fix/datetime-time-serializable branch from a4fbe65 to 3b00367 Compare February 25, 2025 08:25
@surister surister requested review from amotl and seut February 25, 2025 09:02
@surister
Copy link
Copy Markdown
Contributor Author

Docs failing with https://cratedb.com/docs/crate/crash/en/latest/objects.inv < Error 503 first byte timeout

Comment thread tests/client/test_http.py Outdated
@amotl amotl removed their request for review June 18, 2025 13:14
@bgunebakan bgunebakan self-assigned this Jun 1, 2026
Comment thread tests/testing/test_layer.py
@bgunebakan bgunebakan requested review from amotl, mfussenegger and seut June 2, 2026 12:43
@amotl amotl removed their request for review June 2, 2026 13:38
Copy link
Copy Markdown
Member

@seut seut left a comment

Choose a reason for hiding this comment

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

👍 thx!

@bgunebakan bgunebakan merged commit df4d991 into main Jun 2, 2026
17 checks passed
@bgunebakan bgunebakan deleted the fix/datetime-time-serializable branch June 2, 2026 14:20
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.

TypeError: Type is not JSON serializable: datetime.time

4 participants