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

Add tests for Temporal.Instant.p*.subtract #3431

Merged
merged 2 commits into from
Mar 14, 2022
Merged

Conversation

ptomato
Copy link
Contributor

@ptomato ptomato commented Mar 11, 2022

This was originally @FrankYFTang's PR #3076 which was closed without being merged. I'm following up as promised. I rebased it, added assertion messages, did some reformatting, and combined with some existing tests.

(Philip, March 2022: I rebased Frank's original PR tc39#3076, did some
reformatting, removed duplicate tests, and combined with some existing
tests.)
This was originally Frank's PR tc39#3075. I rebased it, added assertion
messages, did some reformatting, and combined with some existing tests.
@ptomato
Copy link
Contributor Author

ptomato commented Mar 12, 2022

I've included #3075 in here as well.

@rwaldron rwaldron merged commit 7d5b0ec into tc39:main Mar 14, 2022
@ptomato ptomato deleted the follow-up-3076 branch March 14, 2022 20:33
ptomato added a commit to tc39/proposal-temporal that referenced this pull request Mar 14, 2022
ptomato added a commit to tc39/proposal-temporal that referenced this pull request Mar 14, 2022
Ms2ger pushed a commit to tc39/proposal-temporal that referenced this pull request Mar 15, 2022
12wrigja pushed a commit to 12wrigja/temporal-polyfill that referenced this pull request Jun 28, 2022
See tc39/test262#3431

UPSTREAM_COMMIT=ef3f004275544a11644f0975bf48ac898b660ee9
12wrigja pushed a commit to 12wrigja/temporal-polyfill that referenced this pull request Jun 28, 2022
See tc39/test262#3431

UPSTREAM_COMMIT=ef3f004275544a11644f0975bf48ac898b660ee9
12wrigja pushed a commit to js-temporal/temporal-polyfill that referenced this pull request Jul 6, 2022
See tc39/test262#3431

UPSTREAM_COMMIT=ef3f004275544a11644f0975bf48ac898b660ee9
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.

3 participants