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

docs: Fix dead links #369

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

AiyionPrime
Copy link
Contributor

@AiyionPrime AiyionPrime commented Jul 25, 2024

@kwinz This intends to resolve #295.

@locka99 Everything but the broken web-client sample is documented properly, as its old actix version features structs not present in the more recent one documented for the main crate.
cargo doc does not handle different major versions with breaking changes clever enough.

This PR consists of atomic conventional commits (which could be cleanly rebased onto master), so the workaround (40ce6b6a75b6c79fb16be75a4f9e216cda2f8e92) can be reverted easily once #291 has been resolved.

@AiyionPrime AiyionPrime force-pushed the ci/deadlinks branch 5 times, most recently from 31848b3 to 41cef1e Compare July 26, 2024 09:27
@AiyionPrime
Copy link
Contributor Author

I disliked the idea of fetching cargo-deadlinks via wget, so I've swapped that part for cargo-binstall.
This is ready for review.

@AiyionPrime

This comment was marked as outdated.

@AiyionPrime AiyionPrime marked this pull request as ready for review July 26, 2024 09:38
@AiyionPrime AiyionPrime force-pushed the ci/deadlinks branch 2 times, most recently from ec24cd7 to 1815a8c Compare July 30, 2024 14:11
Use cargo-binstall to avoid building cargo-deadcode from source in CI.
@AiyionPrime
Copy link
Contributor Author

Rebased onto master, dropped the workaround for the webclient example.

@AiyionPrime
Copy link
Contributor Author

@locka99 is there anything left to do here?

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.

Broken docs.rs documentation. How to do a simple one time read without subscription?
1 participant