Skip to content

Editorial: Cleanup initialize date time format #602

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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 3 commits into from
Closed

Editorial: Cleanup initialize date time format #602

wants to merge 3 commits into from

Conversation

gibson042
Copy link
Contributor

  • use standard conventions
  • avoid alias redefintion
  • simplify {date,time}Style vs. explicit component checking

@gibson042 gibson042 mentioned this pull request Aug 17, 2021
Merged
@gibson042 gibson042 requested review from ryzokuken and sffc January 6, 2022 00:43
@sffc sffc removed their request for review January 9, 2022 07:17
@gibson042 gibson042 mentioned this pull request Mar 17, 2022
Closed
@gibson042 gibson042 force-pushed the 2021-08-cleanup-InitializeDateTimeFormat branch 2 times, most recently from eeacb6d to a504032 Compare March 18, 2022 16:23
@gibson042 gibson042 force-pushed the 2021-08-cleanup-InitializeDateTimeFormat branch from a504032 to 0836725 Compare March 21, 2022 01:32
leobalter added a commit that referenced this pull request Mar 28, 2022
@leobalter
Squashed commit of the following:
61ba27f 
commit f18b054
Merge: 0836725 233d29c
Author: Leo Balter <[email protected]>
Date:   Mon Mar 28 01:48:44 2022 -0700

    Merge branch 'master' into 2021-08-cleanup-InitializeDateTimeFormat

commit 0836725
Author: Richard Gibson <[email protected]>
Date:   Tue Aug 17 18:55:23 2021 -0400

    Editorial: Simplify {date,time}Style vs. explicit component checking

commit a1f6ff3
Author: Richard Gibson <[email protected]>
Date:   Tue Aug 17 18:46:55 2021 -0400

    Editorial: Update InitializeDateTimeFormat to use more standard conventions

Closes #602
@leobalter leobalter mentioned this pull request Mar 28, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ryzokuken ryzokuken Awaiting requested review from ryzokuken

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@gibson042 @leobalter