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

Update and fix Time documentation #54340

Merged
merged 1 commit into from
Oct 29, 2021
Merged

Conversation

aaronfranke
Copy link
Member

Follow-up to #49123 and #54256.

  • Fix bogus docs in get_datetime_string_from_system (my fault, bogus since Add a Time singleton #49123).
  • Fix and clarify statements about UTC added in Docs: Add some notes about timezones and UTC to Time #54256. "Unix epoch assumes UTC" as a blanket statement is false. Time's conversion methods are timezone-agnostic, and when getting information from the system there is usually a utc bool.
  • Fix some style issues with time methods in OS having parameters named utc instead of p_utc.

doc/classes/Time.xml Outdated Show resolved Hide resolved
@mhilbrunner
Copy link
Member

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants