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

Remove no-longer supported private tag #2028

Closed

Conversation

FrankYFTang
Copy link
Contributor

Per tc39/proposal-intl-locale#66
private tags are no longer supported. Remove them from tests.

…ebkit/webkit.git at sha 801a7d15df on Fri Dec 21 2018 19:18:17 GMT+0000 (Coordinated Universal Time)
…revision sha's from export and changed files.

    sourceRevisionAtLastExport: 801a7d15df targetRevisionAtLastExport: 8f629cd
…sha 89eb451c on Fri Dec 21 2018 19:21:03 GMT+0000 (Coordinated Universal Time)
…'s from export and changed files.

    sourceRevisionAtLastExport: 89eb451c targetRevisionAtLastExport: 7519e1b
@FrankYFTang
Copy link
Contributor Author

FrankYFTang commented Jan 9, 2019

@FrankYFTang FrankYFTang changed the title Remove no-longer supported privae tag Remove no-longer supported private tag Jan 9, 2019
@FrankYFTang
Copy link
Contributor Author

based on discussion with @littledan I will change this PR to testing these same locale but with different expectation instead. Stay tune.

leobalter and others added 19 commits January 11, 2019 13:44
Update same-line-async-gen-private-field-usage.js
…est262-automation-export-8f629cd87f

Import test changes from JavaScriptCore
…ation-export-7519e1b462

Import test changes from V8
Move proposals that have been incorporated into the standard to the
corresponding section.
 Update rs-private-getter-alt.case
Add missing features to private elements tests
Previously, the early error prohibiting duplicate entries in
UniqueFormalParameters was only tested in terms of async functions. In
one case, this was misattributed to UniqeFormalParameters and only
enforced for strict mode code.

Extend coverage to the other function-creating productions which include
UniqueFormalParameters (i.e. method definitions and non-async arrow
functions), and update the existing tests to more accurately describe
the source of the error.
This is due to the fact we now only accept Unicode Locale Identifier in
UTS 35.
@FrankYFTang
Copy link
Contributor Author

I totally messed up this PR. I will close it and open a new one

@FrankYFTang FrankYFTang deleted the rmPrivateFromlikely-subtags branch January 24, 2019 18:41
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.

7 participants