Skip to content

Conversation

@JLHwung
Copy link
Contributor

@JLHwung JLHwung commented Dec 10, 2020

Q                       A
Minor: New Feature? Yes
Tests Added + Pass? Yes
License MIT

Note that this PR only add supports to optional-chaining and nullish-coalescing. Support of this.#x ??= 1 and this?.#x is deferred because it will depend on other class properties assumptions implementation.

@JLHwung JLHwung added PR: New Feature 🚀 A type of pull request used for our changelog categories area: assumptions labels Dec 10, 2020
@babel-bot
Copy link
Collaborator

babel-bot commented Dec 10, 2020

Build successful! You can test your changes in the REPL here: https://babeljs.io/repl/build/35013/

@JLHwung JLHwung force-pushed the assumption/noDocumentAll branch from c15e16d to a46ae47 Compare December 10, 2020 21:07
@codesandbox-ci
Copy link

codesandbox-ci bot commented Dec 10, 2020

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit e83ae87:

Sandbox Source
babel-repl-custom-plugin Configuration
babel-plugin-multi-config Configuration


export default declare((api, { loose = false }) => {
api.assertVersion(7);
const noDocumentAll = api.assumption("noDocumentAll") ?? loose;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Are we switching all of these to assumption then lose?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, I'll update the other PRs

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The assumption, when unset, is undefined so any configured assumptions overwrites the loose config.

@nicolo-ribaudo
Copy link
Member

Can you rebase this on feat-7.13.0/babel-core-features?

@JLHwung JLHwung force-pushed the assumption/noDocumentAll branch from a46ae47 to e83ae87 Compare December 11, 2020 23:32
@JLHwung JLHwung changed the base branch from nicolo-ribaudo/assumptions to feat-7.13.0/babel-core-features December 11, 2020 23:32
@nicolo-ribaudo nicolo-ribaudo merged commit 2269430 into babel:feat-7.13.0/babel-core-features Dec 11, 2020
@nicolo-ribaudo nicolo-ribaudo deleted the assumption/noDocumentAll branch December 11, 2020 23:50
nicolo-ribaudo pushed a commit to nicolo-ribaudo/babel that referenced this pull request Jan 12, 2021
nicolo-ribaudo pushed a commit that referenced this pull request Jan 24, 2021
nicolo-ribaudo pushed a commit that referenced this pull request Jan 24, 2021
nicolo-ribaudo pushed a commit to nicolo-ribaudo/babel that referenced this pull request Feb 4, 2021
nicolo-ribaudo pushed a commit to nicolo-ribaudo/babel that referenced this pull request Feb 10, 2021
nicolo-ribaudo pushed a commit to nicolo-ribaudo/babel that referenced this pull request Feb 11, 2021
nicolo-ribaudo added a commit that referenced this pull request Feb 16, 2021
- `mutableTemplateObject` and `ignoreToPrimitiveHint` (#12408)
- `setClassMethods` (#12407)
- `setComputedProperties` (#12490)
- `ignoreFunctionLength` (#12491)
- `noDocumentAll` (#12481)
- `iterableIsArray` and `arrayLikeIsIterable` (#12489)
- `pureGetters` (#12504)
- `skipForOfIteratorClosing` (#12496)
- `objectRestNoSymbols`, `setSpreadProperties` and `pureGetters` (#12505)
- `noNewArrows` (#12613, #12793)
- `setPublicClassFields` and `privateFieldsAsProperties` (#12497)
- `constantReexports` and `enumerableModuleMeta` (#12618)
- `constantSuper`, `superIsCallableConstructor` and `noClassCalls` (#12726)

Co-authored-by: Justin Ridgewell <[email protected]>
Co-authored-by: Huáng Jùnliàng <[email protected]>
nicolo-ribaudo added a commit that referenced this pull request Feb 16, 2021
- `mutableTemplateObject` and `ignoreToPrimitiveHint` (#12408)
- `setClassMethods` (#12407)
- `setComputedProperties` (#12490)
- `ignoreFunctionLength` (#12491)
- `noDocumentAll` (#12481)
- `iterableIsArray` and `arrayLikeIsIterable` (#12489)
- `pureGetters` (#12504)
- `skipForOfIteratorClosing` (#12496)
- `objectRestNoSymbols`, `setSpreadProperties` and `pureGetters` (#12505)
- `noNewArrows` (#12613, #12793)
- `setPublicClassFields` and `privateFieldsAsProperties` (#12497)
- `constantReexports` and `enumerableModuleMeta` (#12618)
- `constantSuper`, `superIsCallableConstructor` and `noClassCalls` (#12726)

Co-authored-by: Justin Ridgewell <[email protected]>
Co-authored-by: Huáng Jùnliàng <[email protected]>
nicolo-ribaudo added a commit that referenced this pull request Feb 21, 2021
- `mutableTemplateObject` and `ignoreToPrimitiveHint` (#12408)
- `setClassMethods` (#12407)
- `setComputedProperties` (#12490)
- `ignoreFunctionLength` (#12491)
- `noDocumentAll` (#12481)
- `iterableIsArray` and `arrayLikeIsIterable` (#12489)
- `pureGetters` (#12504)
- `skipForOfIteratorClosing` (#12496)
- `objectRestNoSymbols`, `setSpreadProperties` and `pureGetters` (#12505)
- `noNewArrows` (#12613, #12793)
- `setPublicClassFields` and `privateFieldsAsProperties` (#12497)
- `constantReexports` and `enumerableModuleMeta` (#12618)
- `constantSuper`, `superIsCallableConstructor` and `noClassCalls` (#12726)

Co-authored-by: Justin Ridgewell <[email protected]>
Co-authored-by: Huáng Jùnliàng <[email protected]>
@github-actions github-actions bot added the outdated A closed issue/PR that is archived due to age. Recommended to make a new issue label Mar 13, 2021
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 13, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area: assumptions outdated A closed issue/PR that is archived due to age. Recommended to make a new issue PR: New Feature 🚀 A type of pull request used for our changelog categories Spec: Nullish Coalescing Operator Spec: Optional Chaining

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants