Skip to content

Update README.md#482

Open
kroggen wants to merge 10 commits into
andpor:masterfrom
kroggen:patch-1
Open

Update README.md#482
kroggen wants to merge 10 commits into
andpor:masterfrom
kroggen:patch-1

Conversation

@kroggen

@kroggen kroggen commented Jul 21, 2021

Copy link
Copy Markdown

Make the readme a little more clear

@kroggen

kroggen commented Aug 24, 2021

Copy link
Copy Markdown
Author

Can someone review this?

The current README is bad formatted, not clear to understand

@andpor

andpor commented Oct 1, 2021

Copy link
Copy Markdown
Owner

@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

kroggen commented Oct 1, 2021

Copy link
Copy Markdown
Author

@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

andpor commented Oct 1, 2021

Copy link
Copy Markdown
Owner

@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

kroggen commented Oct 2, 2021

Copy link
Copy Markdown
Author

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