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

Normative: Remove order check in PartitionDateTimeRangePattern #701

Merged
merged 1 commit into from
Jul 25, 2022

Conversation

FrankYFTang
Copy link
Contributor

Address #692

@FrankYFTang FrankYFTang changed the title Remove order check in PartitionDateTimeRangePattern Normative: Remove order check in PartitionDateTimeRangePattern Jul 8, 2022
@FrankYFTang
Copy link
Contributor Author

@FrankYFTang
Copy link
Contributor Author

we discuss this in 2022-07-20 TC39 on 1pm-1:30pm PST and got consensus to merge it. Please approve andmerge it.

FrankYFTang added a commit to FrankYFTang/test262 that referenced this pull request Jul 21, 2022
In 2022-07-20 TC39 we decide to take 
tc39/ecma402#701 and not check the order of the x and y 
see https://docs.google.com/presentation/d/1UUvbf3FFu9PGtrPAKPdMad9DZuVFLIvkAsAxyJZyvxM/ for details
@FrankYFTang
Copy link
Contributor Author

test262 change in tc39/test262#3609

@FrankYFTang
Copy link
Contributor Author

@gibson042 @ryzokuken I don't have the permission to merge, please click the Merge Pull Request bottom for me.

ptomato pushed a commit to tc39/test262 that referenced this pull request Jul 21, 2022
In 2022-07-20 TC39 we decide to take 
tc39/ecma402#701 and not check the order of the x and y 
see https://docs.google.com/presentation/d/1UUvbf3FFu9PGtrPAKPdMad9DZuVFLIvkAsAxyJZyvxM/ for details
@ryzokuken ryzokuken merged commit 769df4b into master Jul 25, 2022
@ryzokuken ryzokuken deleted the FrankYFTang-patch-2 branch July 25, 2022 13:50
moz-v2v-gh pushed a commit to mozilla/gecko-dev that referenced this pull request Jul 27, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants