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

added lychee config and workflow, fix #718, fix #722 #721

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

Conversation

Arteiii
Copy link
Contributor

@Arteiii Arteiii commented Jan 8, 2025

  • added link check workflow
  • added lychee config file
  • fixed xtask exit code

fixed xtask exit code
@Arteiii Arteiii marked this pull request as draft January 8, 2025 23:21
@Arteiii
Copy link
Contributor Author

Arteiii commented Jan 9, 2025

Should now work with github cache, requires access to prs to submit comments (e.g.: Arteiii#5 (comment))

Fixed some broken links, others require src update as they broke due to breaking changes in the crate (#719)

@Arteiii Arteiii marked this pull request as ready for review January 9, 2025 13:16
- CONTRIBUTING.md (removed link as its not required and no longer available)
- src/encoding/string/url-encode.md (updated form_urlencoded is not a part of url its re-exported)
- README.md (removed travis CI link [404](https://www.travis-ci.com/rust-lang-nursery/rust-cookbook))
- src/development_tools/debugging/config_log/log-custom.md (log4rs::config::Config is a re-export from log4rs::config::runtime::Config)
- src/web/scraping/broken.md (tokio::spawn to tokio::task::spawn)
- src/web/clients/api/rate-limited.md (removed file rust src broken links broken and is not used or displayed on the current cookbook build)
- src/cryptography/hashing/hmac.md (updated description, added tag type hint, and updated links)
@Arteiii Arteiii changed the title added lychee config and workflow, fixes #718 (partially) added lychee config and workflow, fix #718, fix #722 Jan 9, 2025
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.

1 participant