Skip to content
This repository was archived by the owner on Dec 13, 2023. It is now read-only.

Deprecate Roact.reconcile in favor of Roact.update in master #194

Merged
merged 2 commits into from
Apr 27, 2019

Conversation

LPGhatguy
Copy link
Contributor

@LPGhatguy LPGhatguy commented Apr 26, 2019

As a step to trying to make user's code more compatible in preparation for new-reconciler merging back in, this PR deprecates Roact.reconcile in favor of Roact.update, which is more clear.

Roact.reconcile was already deprecated in the new-reconciler branch awhile ago.

Checklist before submitting:

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.2%) to 89.875% when pulling 420f6dd on deprecate-reconcile into f1843c1 on master.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.2%) to 89.875% when pulling 420f6dd on deprecate-reconcile into f1843c1 on master.

Copy link
Contributor

@ZoteTheMighty ZoteTheMighty left a comment

Choose a reason for hiding this comment

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

Looks good.

Did you want to me to update the docs separately?

@LPGhatguy
Copy link
Contributor Author

Yeah, let's fix up the docs when we merge new-reconciler.

@LPGhatguy LPGhatguy merged commit 462ecf7 into master Apr 27, 2019
@LPGhatguy LPGhatguy deleted the deprecate-reconcile branch April 27, 2019 00:45
ZoteTheMighty added a commit that referenced this pull request May 3, 2019
* Remove references to 'primitive' components/elements

* Adjust some wording around kinds of components, fix broken link

* functional component -> function component

* Avoid usage of the verb 'reconcile' where not directly applicable

* Deprecate Roact.reconcile in favor of Roact.update in master (#194)

* Update CHANGELOG

* Remove undocumented APIs `getGlobalConfigValue` and `Element` (#195)

* Roact.reconcile replaced with Roact.update, uses of reconcile changed to update. Occasional mentions of 'the Roact reconciler' have been changed to simply 'Roact'

* Add version info to new apis, more details to validateProps, added section for setGlobalConfig

* Added the 'enable tracebacks' message to other places that try to use element tracebacks

* Change misleading code in Roact.Change example (#196)

* PR comments: mention deprecated names, fix links

* Rename propertyValidation config key to propValidation, add code snippet for enabling it to docs
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants