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

housekeeping: Fix typos and broken links in readme #1708

Merged
merged 2 commits into from
Aug 6, 2018
Merged

housekeeping: Fix typos and broken links in readme #1708

merged 2 commits into from
Aug 6, 2018

Conversation

worldbeater
Copy link
Contributor

@worldbeater worldbeater commented Aug 5, 2018

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
Changes in README.md to make it more friendly to new users.

What is the current behavior? (You can also link to an open issue here)
There are some images not displaying due to broken urls, there are no links to documentation or to "Getting Started" page in old README.md file.

What is the new behavior (if this is a feature change)?
Images are back in town, links to docs, packages and samples are also added. "A Compelling Example" now uses WhenAnyValue instead of WhenAny and x.Value, curly brackets are formatted using .NET Core code of conduct.

What might this PR break?
Nothing, really.

Other information:
Personally I don't like centered text and huge empty space caused by "Table of Contents", so I removed these things.

Add links to documentation, reduce empty space
@worldbeater worldbeater requested a review from a team August 5, 2018 19:49
@glennawatson glennawatson added this to the vNext milestone Aug 6, 2018
@RLittlesII RLittlesII merged commit 80e7cd9 into reactiveui:master Aug 6, 2018
glennawatson pushed a commit that referenced this pull request Mar 23, 2019
* housekeeping: Fix typos and broken links in readme

Add links to documentation, reduce empty space

* housekeeping: add fancy emojis
@lock lock bot locked and limited conversation to collaborators Jun 25, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants