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

Fix README.md to match the files linked just below. #86

Closed
wants to merge 1 commit into from

Conversation

adetaylor
Copy link
Collaborator

I was a bit confused about how this snippet in README.md matched the .h and .cc files below. I figured it out as soon as I looked at the real main.rs but here's a tiny PR to keep them in sync.

Copy link
Owner

@dtolnay dtolnay left a comment

Choose a reason for hiding this comment

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

Thanks for the PR!

I think I would prefer to keep this as originally written. This is the first code snippet in the readme, and so I would prefer to keep it as focused on the important concepts as possible. There is lots more detail filled in by the links for readers who make it that far.

@dtolnay dtolnay closed this Apr 3, 2020
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