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

Jump to bottom

emit error on destructuring of rest property #29609

Merged

RyanCavanaugh merged 1 commit into microsoft:master from ajafff:rest-property-destructuring

Apr 25, 2019

Contributor

ajafff commented Jan 27, 2019

Fixes: #26005


          emit error on destructuring of rest property

b328de6

Fixes: microsoft#26005

ajafff commented

View reviewed changes

tests/baselines/reference/restPropertyWithBindingPattern.errors.txt

+              ==== tests/cases/conformance/es6/destructuring/restPropertyWithBindingPattern.ts (7 errors) ====
+                  ({...{}} = {});
+                       ~~
+              !!! error TS2501: A rest element cannot contain a binding pattern.

Contributor Author

ajafff Jan 27, 2019

"rest element" is not quite correct as it's actually a "rest property", but this is what's used consistently in all other diagnostics

ajafff force-pushed the rest-property-destructuring branch from 2245ed4 to b328de6 Compare

January 27, 2019 17:45

RyanCavanaugh merged commit b45df89 into microsoft:master

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment