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

Reference for stage 3 temporal #37344

Merged
merged 46 commits into from
Jan 20, 2025
Merged

Reference for stage 3 temporal #37344

merged 46 commits into from
Jan 20, 2025

Conversation

Josh-Cena
Copy link
Member

@Josh-Cena Josh-Cena commented Dec 23, 2024

#27745

Far from ready for review, and I'll work on this alone for a while

It is now ready for review! It's extremely big but I think it's more worthwhile getting it in as a whole. I`m already leaving the guide to a second PR.

For reviewers (both editorial and technical): I've made a spreadsheet here: https://docs.google.com/spreadsheets/d/1RrRHL0m08R4V4WtuWBbSWr9a1gO4RiIaH42WrPv_Uhs/edit Welcoming anyone on the team for editorial reviews! Technical review from the champion group is greatly appreciated too. It seems we have roughly 4 weeks to make this happen.

@github-actions github-actions bot added Content:JS JavaScript docs size/xl [PR only] >1000 LoC changed labels Dec 23, 2024
Copy link
Contributor

github-actions bot commented Dec 23, 2024

Preview URLs (282 pages)
External URLs (41)

URL: /en-US/docs/Web/JavaScript/Reference/Global_Objects/JSON
Title: JSON


URL: /en-US/docs/Web/JavaScript/Reference/Global_Objects/Temporal
Title: Temporal


URL: /en-US/docs/Web/JavaScript/Reference/Global_Objects/Temporal/PlainDate
Title: Temporal.PlainDate


URL: /en-US/docs/Web/JavaScript/Reference/Global_Objects/Temporal/PlainDate/day
Title: Temporal.PlainDate.prototype.day


URL: /en-US/docs/Web/JavaScript/Reference/Global_Objects/Temporal/Instant
Title: Temporal.Instant


URL: /en-US/docs/Web/JavaScript/Reference/Global_Objects/Temporal/PlainYearMonth
Title: Temporal.PlainYearMonth


URL: /en-US/docs/Web/JavaScript/Reference/Global_Objects/Temporal/ZonedDateTime
Title: Temporal.ZonedDateTime


URL: /en-US/docs/Web/JavaScript/Reference/Global_Objects/Temporal/ZonedDateTime/day
Title: Temporal.ZonedDateTime.prototype.day


URL: /en-US/docs/Web/JavaScript/Reference/Global_Objects/Temporal/ZonedDateTime/add
Title: Temporal.ZonedDateTime.prototype.add()


URL: /en-US/docs/Web/JavaScript/Reference/Global_Objects/Temporal/ZonedDateTime/since
Title: Temporal.ZonedDateTime.prototype.since()


URL: /en-US/docs/Web/JavaScript/Reference/Global_Objects/Temporal/PlainDateTime
Title: Temporal.PlainDateTime


URL: /en-US/docs/Web/JavaScript/Reference/Global_Objects/Temporal/PlainMonthDay
Title: Temporal.PlainMonthDay


URL: /en-US/docs/Web/JavaScript/Reference/Global_Objects/Temporal/Duration
Title: Temporal.Duration

TRUNCATED!

@Josh-Cena Josh-Cena marked this pull request as ready for review January 3, 2025 09:04
@Josh-Cena Josh-Cena requested a review from a team as a code owner January 3, 2025 09:04
@Josh-Cena Josh-Cena requested review from chrisdavidmills and hamishwillee and removed request for a team January 3, 2025 09:04
Copy link
Contributor

This pull request has merge conflicts that must be resolved before it can be merged.

@github-actions github-actions bot added the merge conflicts 🚧 [PR only] label Jan 14, 2025
@github-actions github-actions bot removed the merge conflicts 🚧 [PR only] label Jan 14, 2025
Copy link
Member

@bsmth bsmth left a comment

Choose a reason for hiding this comment

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

Couple of comments for Conversion group, nearly there

@github-actions github-actions bot added the merge conflicts 🚧 [PR only] label Jan 16, 2025
Copy link
Contributor

This pull request has merge conflicts that must be resolved before it can be merged.

@github-actions github-actions bot removed the merge conflicts 🚧 [PR only] label Jan 16, 2025
Copy link
Member

@bsmth bsmth left a comment

Choose a reason for hiding this comment

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

Minor comments in Serialization group

Copy link
Member

@bsmth bsmth left a comment

Choose a reason for hiding this comment

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

Last ones on Temporal.Duration

Copy link
Member

@bsmth bsmth left a comment

Choose a reason for hiding this comment

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

Amazing work. Some comments for you to check, but I'm more than happy to leave a +1, well done 👏🏻

@Josh-Cena
Copy link
Member Author

I plan to merge it some time next week, because I currently have another PR that depends on the Intl changes, and I also need to work on the guide. People are encouraged to leave comments as early as possible, but we can definitely accept follow-up PRs, preferably for bug fixes, especially from the champion group.

@Josh-Cena
Copy link
Member Author

@printfn Done, thanks for pointing out! I plan to merge it today, so if you have or don't have more comments please feel free to send them.

@Josh-Cena Josh-Cena changed the title [new-ci] Reference for stage 3 temporal Reference for stage 3 temporal Jan 20, 2025
@Josh-Cena
Copy link
Member Author

I'm going to go ahead and merge it. Thanks everyone! I understand that this is not the final version, and we will surely continue to iterate on it.

@Josh-Cena Josh-Cena merged commit a4e9bce into mdn:main Jan 20, 2025
9 checks passed
@Josh-Cena Josh-Cena deleted the temporal branch January 20, 2025 22:28
@justingrant
Copy link
Contributor

Thanks so much @Josh-Cena for making this happen!

Also grateful for @meyerweb for all the work you did on the docs previously, and in helping to review this PR.

Developers worldwide are going to be thankful. 🙏

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Content:JS JavaScript docs size/xl [PR only] >1000 LoC changed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants