Skip to content
This repository has been archived by the owner on Mar 1, 2024. It is now read-only.

Update READMEs to make refs clearer #138

Merged
merged 3 commits into from
Jan 27, 2020
Merged

Conversation

taraojo
Copy link
Contributor

@taraojo taraojo commented Jan 24, 2020

With the change from using innerRef to just using ref, I've opened this PR to make the docs a little clearer.

@nemobot nemobot added the WIP label Jan 24, 2020
Copy link
Member

@giogonzo giogonzo left a comment

Choose a reason for hiding this comment

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

Thanks @taraojo!

Left a minor inline comments, looks good for the rest

README.md Outdated
@@ -17,6 +17,9 @@ ReactDOM.renderComponent(
```
npm install --save react-autosize-textarea
```
```
yarn add --dev react-autosize-textarea
Copy link
Member

Choose a reason for hiding this comment

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

why --dev? Should be a "prod" dependency

Copy link
Contributor Author

Choose a reason for hiding this comment

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

whoops, thanks @giogonzo - I'll remove that 🙂

Tara Ojo and others added 2 commits January 26, 2020 21:09
@nemobot nemobot added in review and removed WIP labels Jan 27, 2020
@giogonzo giogonzo merged commit 46b91a4 into buildo:master Jan 27, 2020
@nemobot
Copy link

nemobot commented Jan 27, 2020

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