Skip to content

fix: use timestamp() for IndexEntry to git2::IndexEntry conversion#201

Merged
seokju-na merged 1 commit into
toss:mainfrom
xxxxxxjun:fix-index-timestamp
May 7, 2026
Merged

fix: use timestamp() for IndexEntry to git2::IndexEntry conversion#201
seokju-na merged 1 commit into
toss:mainfrom
xxxxxxjun:fix-index-timestamp

Conversation

@xxxxxxjun
Copy link
Copy Markdown
Contributor

Closes #200.

Replaces chrono::Timelike::second() (0–59) with DateTime::timestamp() (Unix timestamp), and nanosecond() with timestamp_subsec_nanos() to avoid leap-second overflow.

@xxxxxxjun xxxxxxjun requested a review from seokju-na as a code owner May 4, 2026 23:41
@vercel
Copy link
Copy Markdown

vercel Bot commented May 4, 2026

@xxxxxxjun is attempting to deploy a commit to the Toss Team on Vercel.

A member of the Team first needs to authorize it.

Copy link
Copy Markdown
Collaborator

@seokju-na seokju-na left a comment

Choose a reason for hiding this comment

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

Thanks!

@seokju-na seokju-na merged commit da96e9e into toss:main May 7, 2026
42 of 44 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.

Wrong chrono method in IndexEntry → git2::IndexEntry truncates ctime/mtime

2 participants