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

[C-METADATA] package.documentation should be optional #117

Closed
behnam opened this Issue Aug 5, 2017 · 1 comment

Comments

Projects
None yet
2 participants
@behnam
Copy link
Contributor

behnam commented Aug 5, 2017

The package.documentation now has a reasonable default value, using docs.rs, meaning that it actually makes sense to not have it specifically set for most crates.

Therefore, it makes sense to update C-METADATA to mention this default value and ask the config to be set only if it's not the default docs.rs link. This exception would be similar to the package.homepage config.

@dtolnay dtolnay added edit clarification and removed edit labels Sep 18, 2017

@dtolnay

This comment has been minimized.

Copy link
Member

dtolnay commented Oct 29, 2017

Thanks! I removed the docs.rs guideline in b7a831e and explained that the documentation metadata in Cargo.toml is optional in 17073b7.

@dtolnay dtolnay closed this Oct 29, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.