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

CI: https://rust-random.github.io/rand is outdated #1066

Closed
dhardy opened this issue Nov 18, 2020 · 7 comments · Fixed by #1073
Closed

CI: https://rust-random.github.io/rand is outdated #1066

dhardy opened this issue Nov 18, 2020 · 7 comments · Fixed by #1073

Comments

@dhardy
Copy link
Member

dhardy commented Nov 18, 2020

Fix publishing this. Or we could drop it since it may be less useful now.

Also consider migrating to GitHub Actions (if it's worth the effort).

@newpavlov
Copy link
Member

+1 for migrating to GitHub Actions, in RustCrypto's case it was a pretty nice experience.

@dhardy dhardy mentioned this issue Dec 2, 2020
19 tasks
@vks
Copy link
Collaborator

vks commented Dec 2, 2020

What's wrong with just using docs.rs?

@dhardy
Copy link
Member Author

dhardy commented Dec 2, 2020

Only that we still link this in the readme. Our self-hosted docs in theory have better cross-crate linking but that shouldn't be significant.

@newpavlov
Copy link
Member

It looks like Travis will not be providing free CI for OSS projects any longer.

@dhardy
Copy link
Member Author

dhardy commented Dec 8, 2020

FYI I'm working on this. No choice now it seems.

@vks
Copy link
Collaborator

vks commented Dec 8, 2020

Are you working on moving to GitHub actions or something else?

@dhardy
Copy link
Member Author

dhardy commented Dec 14, 2020

I can confirm this is fixed now — e.g. the recently added Geometric distribution (#1062) is now documented.

One bug: the "API master" doc link from the rand README is broken. I'll fix soon.

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 a pull request may close this issue.

3 participants