Skip to content

Conversation

@gnapse
Copy link
Contributor

@gnapse gnapse commented Feb 1, 2018

Fixes #47

README.md Outdated
<br />

> This branch is **unstable** and is in **active development**.
> This branch is **unstable** and is in **active development**.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Did you remove double space in the end?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Oh sorry, it probably is my editor configured to remove trailing whitespace. I can add it back if you want.

Copy link
Owner

@renatorib renatorib Feb 1, 2018

Choose a reason for hiding this comment

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

Two trailing whitespaces in markdown is a line break, it's make difference. 😢

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ooops, did not know that. I'll fix it and fix my editor settings. Sorry.

@renatorib renatorib self-requested a review February 1, 2018 18:24
@TrySound
Copy link
Collaborator

TrySound commented Feb 1, 2018

@gnapse There's a lot more. Look at diff.

@gnapse
Copy link
Contributor Author

gnapse commented Feb 1, 2018

Think I fixed them all now. Sorry. First time I see this markdown feature.

@renatorib
Copy link
Owner

I'll review & merge this soon

@renatorib renatorib merged commit 3b44927 into renatorib:master Feb 14, 2018
@gnapse gnapse deleted the children-props-name-conflict branch February 14, 2018 13:10
@renatorib
Copy link
Owner

Thanks @gnapse :)

@gnapse
Copy link
Contributor Author

gnapse commented Feb 14, 2018

Always happy to contribute to open source.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants