Skip to content

Comments

Use datetime.UTC for Python 3.13+ compatibility#37

Merged
babakjahan merged 2 commits intoimprovement/1028from
copilot/sub-pr-36
Nov 4, 2025
Merged

Use datetime.UTC for Python 3.13+ compatibility#37
babakjahan merged 2 commits intoimprovement/1028from
copilot/sub-pr-36

Conversation

Copy link

Copilot AI commented Nov 4, 2025

Addresses review feedback on PR #36 regarding deprecated datetime usage in app/common/http_response/base.py.

Changes

  • Replace timezone.utc with datetime.UTC in BaseResponse timestamp field
  • Remove unused timezone import

The datetime.UTC constant is the modern Python 3.11+ approach and is recommended for projects using Python 3.13+.

# Before
from datetime import datetime, timezone
timestamp: datetime = Field(default_factory=lambda: datetime.now(timezone.utc))

# After  
from datetime import datetime
timestamp: datetime = Field(default_factory=lambda: datetime.now(datetime.UTC))

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Copilot AI mentioned this pull request Nov 4, 2025
@babakjahan babakjahan marked this pull request as ready for review November 4, 2025 23:45
Co-authored-by: babakjahan <5642363+babakjahan@users.noreply.github.com>
Copilot AI changed the title [WIP] Update code to address feedback from PR #36 Use datetime.UTC for Python 3.13+ compatibility Nov 4, 2025
Copilot AI requested a review from babakjahan November 4, 2025 23:47
@babakjahan babakjahan merged commit 02e025c into improvement/1028 Nov 4, 2025
@babakjahan babakjahan deleted the copilot/sub-pr-36 branch November 23, 2025 22:12
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