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

Rename reify to mount and teardown to unmount #85

Merged
merged 7 commits into from
May 10, 2018

Conversation

LPGhatguy
Copy link
Contributor

@LPGhatguy LPGhatguy commented May 9, 2018

Closes #82.

This starts to bring Roact's terminology more into focus, and should get rid of a couple of confusing terms: reification and teardown (which doesn't have a good present-tense verb form).

Now, everywhere can refer to mounting and unmounting as the process.

TODO:

  • Add warnings to ReconcilerCompat
  • Update docs
  • Improve test coverage for deprecation

@coveralls
Copy link

coveralls commented May 9, 2018

Coverage Status

Coverage increased (+0.3%) to 89.813% when pulling 637a3cf on rename-mount-unmount into 8575fc4 on master.

@LPGhatguy LPGhatguy changed the title WIP: Rename reify to mount and teardown to unmount Rename reify to mount and teardown to unmount May 9, 2018
Copy link

@SlartibartfastFjords SlartibartfastFjords 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 to me!

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.

LGTM. You fixed the typo in the change log before I could comment 👍

@LPGhatguy
Copy link
Contributor Author

I force-pushed my change to hide my mistake as well :)

@LPGhatguy LPGhatguy merged commit ddac5ae into master May 10, 2018
@LPGhatguy LPGhatguy deleted the rename-mount-unmount branch May 10, 2018 00:52
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.

4 participants