Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update README.md #482

Open
wants to merge 10 commits into
base: master
Choose a base branch
from
Open

Update README.md #482

wants to merge 10 commits into from

Conversation

kroggen
Copy link

@kroggen kroggen commented Jul 21, 2021

Make the readme a little more clear

@kroggen
Copy link
Author

kroggen commented Aug 24, 2021

Can someone review this?

The current README is bad formatted, not clear to understand

@andpor
Copy link
Owner

andpor commented Oct 1, 2021

@kroggen there seem to be conflicts. Could you please check why and make adjustments in your PR. I will merge it once it is clean. Thanks!

@kroggen
Copy link
Author

kroggen commented Oct 1, 2021

@andpor

Done.

I think that the README is still confusing because it mix instructions for many versions of RN. In my opinion the README should contain instructions for only the last version (with autolinking) and move the instructions for older versions to another file. Fewer people will read them anyway, and it will make the README way more clear.

@andpor
Copy link
Owner

andpor commented Oct 1, 2021

@kroggen thanks a lot. I like your idea. Do you want to take a stab at breaking up the Readme file as you suggested ?

@kroggen
Copy link
Author

kroggen commented Oct 2, 2021

Lets do it in steps. You can merge this one first. Maybe someone else that understands React Native could do it better than me. My world is C :P

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.

2 participants