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

move documentation from README.md to lib.rs #921

Merged
merged 1 commit into from
Feb 12, 2024
Merged

Conversation

Be-ing
Copy link
Contributor

@Be-ing Be-ing commented Jan 28, 2024

so it appears on https://docs.rs and inconsistencies aren't created between README.md and lib.rs

per discussion in #920

The old introductory text in lib.rs has been replaced with the text from README.md and the introductory paragraph revised. Otherwise no major changes to the text have been made.

I will follow this PR with more substantial edits to the text.

so it appears on https://docs.rs and inconsistencies aren't
created between README.md and lib.rs

per discussion in rust-lang#920

The old introductory text in lib.rs has been replaced with
the text from README.md and the introductory paragraph revised.
Otherwise no major changes to the text have been made.
Copy link
Collaborator

@NobodyXu NobodyXu left a comment

Choose a reason for hiding this comment

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

Thanks, it LGTM but I would like feedback from others as well.

@NobodyXu
Copy link
Collaborator

cc @thomcc I would like your review on this PR

@NobodyXu
Copy link
Collaborator

NobodyXu commented Feb 4, 2024

I will merge this before next release to minimize the disruption.

@Be-ing Be-ing mentioned this pull request Feb 12, 2024
@NobodyXu NobodyXu merged commit 243038b into rust-lang:main Feb 12, 2024
18 checks passed
@BlackHoleFox
Copy link
Contributor

potentially both could be kept by using the doc = include_str! trick. I think cc's MSRV is high enough for that now?

@Be-ing
Copy link
Contributor Author

Be-ing commented Feb 13, 2024

The trouble with doc = include_str!("README.md") is that https://github.com/rust-lang/cc-rs will show documentation for unreleased changes. I have been confused before by another library doing that. Linking to https://docs.rs/cc doesn't have that issue; people will only see the documentation for published releases.

@Be-ing Be-ing deleted the doc branch February 13, 2024 01:23
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.

None yet

3 participants