Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: Time std lib #202

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

feat: Time std lib #202

wants to merge 3 commits into from

Conversation

giann
Copy link
Collaborator

@giann giann commented Oct 5, 2023

closes #173

@giann giann linked an issue Oct 5, 2023 that may be closed by this pull request
3 tasks
message: "Could format time"
);

assert(time.timestamp() == 1696491535, message: "Could use timestamp");

Choose a reason for hiding this comment

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

Suggested change
assert(time.timestamp() == 1696491535, message: "Could use timestamp");
assert(time.timestamp() == 1696523935, message: "Could use timestamp");

I don't know if this is correct but it's what I'm seeing on Mac if I dump(time.timestamp()) before this line.

Choose a reason for hiding this comment

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

Oh if it's timezone related... I'm in the Pacific timezone

@giann giann force-pushed the time-lib branch 5 times, most recently from e4f8da5 to 7b88159 Compare October 10, 2023 09:55
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.

Time manipulation std lib
2 participants