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

apply playpen by default #104

Closed
wants to merge 1 commit into from
Closed

Conversation

liigo
Copy link

@liigo liigo commented Feb 9, 2017

The standard library API docs allow us running example code at https://play.rust-lang.org/ (which powered by playpen). But there're only very little amount of crates (i.g. num, frunk, rusqbin) on crates.io have this feature. Most other crate authors don't know about #![doc(html_playground_url = "https://play.rust-lang.org/")], or forget adding it to crate root.

Since rustdoc has --playground-url option (cc rust-lang/rust#37763), it's time to apply playpen by default to all crates.io crates. For crates have their own #![doc(html_playground_url = "URL")], nothing will be changed.

@onur
Copy link
Member

onur commented Feb 9, 2017

Thanks this is awesome! But AFAIK playpen doesn't support "extern crate" definitions. I don't think it is usable for crates in docs.rs or crates.io. An example from rand crate: https://is.gd/B7FUQf

@liigo
Copy link
Author

liigo commented Feb 10, 2017

yes, playpen should solve this issue first. cc rust-lang/rust-playpen#67

@liigo liigo closed this Feb 10, 2017
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