Skip to content

Review support nanosecond timezone#4471

Merged
aziemchawdhary-gs merged 4 commits intofinos:masterfrom
goldmansachs:review-support-nanosecond-timezone
Mar 2, 2026
Merged

Review support nanosecond timezone#4471
aziemchawdhary-gs merged 4 commits intofinos:masterfrom
goldmansachs:review-support-nanosecond-timezone

Conversation

@gs-afcarzero1
Copy link
Copy Markdown
Contributor

What type of PR is this?

  • Bug Fix

What does this PR do / why is it needed ?

  • Allows for serialization of models having DateTime with nanoseconds and timezones.

Which issue(s) this PR fixes:

Fixes #

Other notes for reviewers:

Does this PR introduce a user-facing change?

No

@gs-afcarzero1 gs-afcarzero1 requested a review from a team as a code owner February 25, 2026 10:01
@linux-foundation-easycla
Copy link
Copy Markdown

linux-foundation-easycla Bot commented Feb 25, 2026

CLA Signed

The committers listed above are authorized under a signed CLA.

@github-actions
Copy link
Copy Markdown

Test Results

  1 070 files   -      18    1 070 suites   - 18   3h 4m 33s ⏱️ - 53m 7s
13 629 tests  -      79  13 453 ✔️  -      79  176 💤 ±0  0 ±0 
32 078 runs   - 2 756  31 902 ✔️  - 2 756  176 💤 ±0  0 ±0 

Results for commit 2010fdf. ± Comparison against base commit 7cac070.

@aziemchawdhary-gs aziemchawdhary-gs merged commit 336cbaf into finos:master Mar 2, 2026
26 checks passed
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