Skip to content

Remove quickstart playground link#94

Merged
dhardy merged 2 commits intorust-random:masterfrom
andrewsartor:update-quick-start-playground-link
Jan 24, 2026
Merged

Remove quickstart playground link#94
dhardy merged 2 commits intorust-random:masterfrom
andrewsartor:update-quick-start-playground-link

Conversation

@andrewsartor
Copy link
Contributor

@andrewsartor andrewsartor commented Jan 22, 2026

Summary

Updates Removes the playground link in the quick start page to use the updated sample code. The previously linked playground contained deprecated code which would produce build errors and fail to run.

Addresses #95

@andrewsartor andrewsartor force-pushed the update-quick-start-playground-link branch from fe5bf8a to a97e477 Compare January 22, 2026 13:12
@andrewsartor andrewsartor force-pushed the update-quick-start-playground-link branch from a97e477 to 9709a4a Compare January 22, 2026 13:13
@dhardy
Copy link
Member

dhardy commented Jan 23, 2026

Thanks for the PR.

Note however that the Quick Start code sample is runnable (play button in top right corner), so I think it would be better just to remove the playground link.

@andrewsartor
Copy link
Contributor Author

Thank you for your response. In that case, should playground links be removed from the book generally at this point?

@dhardy
Copy link
Member

dhardy commented Jan 23, 2026

I haven't checked them all, but if the code snippet already has a functional "play" button and is editable, then yes.

@andrewsartor andrewsartor changed the title Update quickstart playground link to use updated demo code Remove quickstart playground link Jan 24, 2026
@andrewsartor
Copy link
Contributor Author

After a quick perusal of the rest of the repo, it looks like this was the only playground link remaining, which I have removed in 297a0e6.

@dhardy dhardy merged commit f58e348 into rust-random:master Jan 24, 2026
2 checks passed
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