Skip to content

fix: fix os.date("!*t", os.time()) will not get UTC date time issue#433

Merged
yuin merged 3 commits into
yuin:masterfrom
ttys3:fix-utc-date-format
May 30, 2023
Merged

fix: fix os.date("!*t", os.time()) will not get UTC date time issue#433
yuin merged 3 commits into
yuin:masterfrom
ttys3:fix-utc-date-format

Conversation

@ttys3

@ttys3 ttys3 commented Apr 12, 2023

Copy link
Copy Markdown
Contributor

Fixes os.date("!*t", os.time()) will not get UTC date time issue

Comment thread baselib_test.go
@ttys3 ttys3 requested a review from yuin May 30, 2023 06:38
@yuin

yuin commented May 30, 2023

Copy link
Copy Markdown
Owner

@ttys3 Sorry for misunderstanding your PR.
I had thought os.Setenv but you used t.Setenv so you do not need to rollback the TZ envvar.
Could you add t.Setenv again?

@ttys3

ttys3 commented May 30, 2023

Copy link
Copy Markdown
Contributor Author

@ttys3 Sorry for misunderstanding your PR. I had thought os.Setenv but you used t.Setenv so you do not need to rollback the TZ envvar. Could you add t.Setenv again?

done. PTAL

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