Fixes issue 10989 where the only the identifier in a member expression that is the value of an object property would be parsed. Removing checkExpressionErrors in parseExprSubscripts results in the subscript also being parsed.
* Fix rest parameters indexing with TypeScript 'this parameter'
The TypeScript [this parameter][0] is a fake parameter and should not be
taken into account when counting the function parameters. This patch
skips it by using the condition taken from the `transform-typescript`
plugin.
Note: since the `transform-typescript` plugin is removing this kind of
parameter, including it before the `transform-parameters` plugin solves
the issue. This patch fixes the issue in other cases.
[0]: https://www.typescriptlang.org/docs/handbook/functions.html#this-parameters
* nit: improve the 'this parameter' detection condition
* improve performance by checking the parameter list length before
accessing it
* simplify the test a bit by using the `isIdentifier` second
parameter
I really don't like this commit, but import() is currently breaking
our publish script.
When "normal" tests we are transpiling import() so that it works
with Jest. We can't do it while publishing because we need to
publish the untranspiled import() so that it can load real .mjs files.
Follow up to #10903
* Added precedence for nullish-coalescing-operator
* Made precedence equal to logical OR
* Renamed the folders made for testing
* Fixed the output test file of nullish-coalescing op
* refactor: replace refShorthandDefaultPos by refExpressionErrors
* fix: duplicate __proto__ keys should be allowed in patterns
* docs: add comments for ExpressionErrors.doubleProto [ci-skip]
* test: add more test for coverage
* fix: disallow private name in object member and TS type elements
* chore: update test262 whitelist
* chore: make flow happy
* Update packages/babel-parser/src/parser/expression.js
Co-Authored-By: Nicolò Ribaudo <nicolo.ribaudo@gmail.com>
* chore: update test fixtures
* Update packages/babel-parser/src/parser/expression.js
Co-Authored-By: Brian Ng <bng412@gmail.com>
* chore: update test fixtures
Co-authored-by: Nicolò Ribaudo <nicolo.ribaudo@gmail.com>
Co-authored-by: Brian Ng <bng412@gmail.com>
* feat: add babel-preset-env to babel-standalone
* chore: copy preset-env-standalone test
* polish: tune warning message when babel-preset-env-standalone is loaded
* chore: add test files
# Conflicts:
# packages/babel-node/package.json
* fix: respect short flags when finding filename index
* fix: use attributeName for kebab-case
* prevent ignored files in out dir
* added includeIgnore cli option
* Help text change
* Update packages/babel-cli/src/babel/options.js
Copy review.
Co-Authored-By: Brian Ng <bng412@gmail.com>
* review comments
* throw error if copyIgnored is used without ignore flag
* check for ignored files
* duplicate pathToPattern fn in babel/cli
* change implementation
* removed ignore option from cliOption
* added test case with ignore in config
* added test case with ignore in config
* review
Co-authored-by: Brian Ng <bng412@gmail.com>
* Add --use-file-extension option to babel-cli
* refactor file extension logic
use existing methods from path module
* Rename use-file-extension -> out-file-extension
* add error message if bot keep- and out-file-extension is used
* add test for error when mixing --keep- and --out-file-extension
* poke travis
* chore: rename test folder
* chore: add test case
* fix: use chrome support data when android is absent
* chore: add type annotation
* chore: add testcases last 2 versions not ie
* chore: bump browerslist to v4.8.2
* chore: update test fixtures
* chore: update test fixtures
