Skip to content

Remove DST notes from gmtime docs#4956

Merged
Jak-MS merged 4 commits into
MicrosoftDocs:mainfrom
TartanLlama:patch-3
Feb 23, 2024
Merged

Remove DST notes from gmtime docs#4956
Jak-MS merged 4 commits into
MicrosoftDocs:mainfrom
TartanLlama:patch-3

Conversation

@TartanLlama
Copy link
Copy Markdown
Contributor

@TartanLlama TartanLlama commented Feb 23, 2024 •
edited
Loading

Remove note about daylight savings time, as these functions convert from time-since-epoch to UTC, neither of which are affected by daylight savings
Fixes #3443

@TartanLlama
Update gmtime-s-gmtime32-s-gmtime64-s.md
3c1f3d5 
Remove note about daylight savings time, as these functions convert from time-since-epoch to UTC, neither of which are affected by daylight savings
@TartanLlama
Update gmtime-gmtime32-gmtime64.md
faf183f
@prmerger-automator
Copy link
Copy Markdown
Contributor

prmerger-automator Bot commented Feb 23, 2024

@TartanLlama : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@prmerger-automator
Copy link
Copy Markdown
Contributor

prmerger-automator Bot commented Feb 23, 2024

@TartanLlama : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@learn-build-service-prod
Copy link
Copy Markdown
Contributor

learn-build-service-prod Bot commented Feb 23, 2024

Learn Build status updates of commit faf183f:

✅ Validation status: passed

File Status Preview URL Details
docs/c-runtime-library/reference/gmtime-gmtime32-gmtime64.md ✅Succeeded
docs/c-runtime-library/reference/gmtime-s-gmtime32-s-gmtime64-s.md ✅Succeeded

For more details, please refer to the build report.

For any questions, please:

@Jak-MS
Copy link
Copy Markdown
Contributor

Jak-MS commented Feb 23, 2024

@TylerMSFT

  • Can you review this PR?
  • IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@prmerger-automator prmerger-automator Bot added the aq-pr-triaged Tracking label for the PR review team label Feb 23, 2024
TylerMSFT
TylerMSFT approved these changes Feb 23, 2024
View reviewed changes
Copy link
Copy Markdown
Collaborator

@TylerMSFT TylerMSFT left a comment

Choose a reason for hiding this comment

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

Thank you, Sy!

@learn-build-service-prod
Copy link
Copy Markdown
Contributor

learn-build-service-prod Bot commented Feb 23, 2024

Learn Build status updates of commit 517b027:

✅ Validation status: passed

File Status Preview URL Details
docs/c-runtime-library/reference/gmtime-gmtime32-gmtime64.md ✅Succeeded
docs/c-runtime-library/reference/gmtime-s-gmtime32-s-gmtime64-s.md ✅Succeeded

For more details, please refer to the build report.

For any questions, please:

@TylerMSFT
Copy link
Copy Markdown
Collaborator

TylerMSFT commented Feb 23, 2024

#sign-off

@Jak-MS Jak-MS merged commit 1fc38cd into MicrosoftDocs:main Feb 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@TylerMSFT TylerMSFT TylerMSFT approved these changes

Assignees

@TylerMSFT TylerMSFT

Labels

aq-pr-triaged Tracking label for the PR review team Change sent to author qualifies-for-auto-merge ready-to-merge ucrt/subsvc visual-cpp/svc

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

docs for gmtime functions have irrelevant notes about daylight savings time

3 participants

@TartanLlama @Jak-MS @TylerMSFT