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

Rename Temporal.now => Temporal.Now #1645

Merged
merged 2 commits into from
Jul 16, 2021
Merged

Conversation

justingrant
Copy link
Collaborator

Fixes #1583 per today's consensus decision in TC39 plenary meeting.

Note that this PR does not include a toStringTag implementation. See #1539.

@codecov
Copy link

codecov bot commented Jul 15, 2021

Codecov Report

Merging #1645 (975be92) into main (b77da58) will not change coverage.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #1645   +/-   ##
=======================================
  Coverage   95.00%   95.00%           
=======================================
  Files          19       19           
  Lines       10726    10726           
  Branches     1716     1716           
=======================================
  Hits        10190    10190           
  Misses        523      523           
  Partials       13       13           
Flag Coverage Δ
test262 72.80% <100.00%> (ø)
tests 90.09% <100.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
polyfill/lib/now.mjs 100.00% <100.00%> (ø)
polyfill/lib/shim.mjs 100.00% <100.00%> (ø)
polyfill/lib/temporal.mjs 95.23% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b77da58...975be92. Read the comment docs.

Copy link
Member

@ljharb ljharb left a comment

Choose a reason for hiding this comment

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

i only reviewed the one spec file; while i despise the capitalized form, this PR seems correct to me.

spec/temporal.html Show resolved Hide resolved
polyfill/lib/calendar.mjs Outdated Show resolved Hide resolved
polyfill/lib/calendar.mjs Outdated Show resolved Hide resolved
polyfill/test/exports.mjs Outdated Show resolved Hide resolved
polyfill/test/now/instant/length.js Show resolved Hide resolved
spec/temporal.html Outdated Show resolved Hide resolved
spec/temporal.html Outdated Show resolved Hide resolved
@Ms2ger Ms2ger merged commit 0097fdd into tc39:main Jul 16, 2021
justingrant added a commit to justingrant/temporal-polyfill that referenced this pull request Jul 16, 2021
Ms2ger pushed a commit to js-temporal/temporal-polyfill that referenced this pull request Jul 16, 2021
webkit-commit-queue pushed a commit to WebKit/WebKit that referenced this pull request Jul 31, 2021
https://bugs.webkit.org/show_bug.cgi?id=228658

Patch by Philip Chimento <[email protected]> on 2021-07-30
Reviewed by Yusuke Suzuki.

See tc39/proposal-temporal#1645. In the July
2021 meeting TC39 achieved consensus to rename Temporal.now to
Temporal.Now since it is a namespace.

* runtime/TemporalNow.cpp: Change Now[@@toStringTag] accordingly.
* runtime/TemporalObject.cpp: Rename now property to Now.

Canonical link: https://commits.webkit.org/240138@main
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@280506 268f45cc-cd09-0410-ab3c-d52691b4dbfc
philn pushed a commit to philn/old-webkit that referenced this pull request Aug 1, 2021
https://bugs.webkit.org/show_bug.cgi?id=228658

Patch by Philip Chimento <[email protected]> on 2021-07-30
Reviewed by Yusuke Suzuki.

See tc39/proposal-temporal#1645. In the July
2021 meeting TC39 achieved consensus to rename Temporal.now to
Temporal.Now since it is a namespace.

* runtime/TemporalNow.cpp: Change Now[@@toStringTag] accordingly.
* runtime/TemporalObject.cpp: Rename now property to Now.

git-svn-id: http://svn.webkit.org/repository/webkit/trunk@280506 268f45cc-cd09-0410-ab3c-d52691b4dbfc
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.

Temporal.now should maybe be Temporal.Now?
3 participants