Skip to content

Notes on Week Numbering - #31

Open
mbarkhau wants to merge 4 commits into
mahmoud:masterfrom
mbarkhau:patch-3
Open

Notes on Week Numbering#31
mbarkhau wants to merge 4 commits into
mahmoud:masterfrom
mbarkhau:patch-3

Conversation

@mbarkhau

Copy link
Copy Markdown

No description provided.

Comment thread entries/overview.md Outdated
@mbarkhau mbarkhau mentioned this pull request Aug 30, 2020
Comment thread entries/overview.md
Comment on lines +82 to +85
* There are at least four common week numbering schemes: The
[ISO][iso-week] and the [US Broadcast][us-week] standards as well
as the output of `%W` and `%U` in many implementations of `strftime`
which produce numbers in the range `[0-52]`.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Suggested change
* There are at least four common week numbering schemes: The
[ISO][iso-week] and the [US Broadcast][us-week] standards as well
as the output of `%W` and `%U` in many implementations of `strftime`
which produce numbers in the range `[0-52]`.
* There are at least four common week numbering schemes, three of which have [strftime][strftime] codes:
* `%U` 00..53, begins Sunday
* `%V` [ISO week][iso-week] 01..53, begins Monday; pair with `%G` for year
* `%W` 00..53, begins Monday
* [US Broadcast week][us-week]

Comment thread entries/overview.md
if it was published on Friday the 1st of January 2021.

[iso-week]: https://en.wikipedia.org/wiki/ISO_week_date
[us-week]: https://en.wikipedia.org/wiki/Broadcast_calendar

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Suggested change
[us-week]: https://en.wikipedia.org/wiki/Broadcast_calendar
[strftime]: https://pubs.opengroup.org/onlinepubs/009696799/functions/strftime.html
[us-week]: https://en.wikipedia.org/wiki/Broadcast_calendar

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