Skip to content

docs: use HTTPS for the astyle and cppcheck links in README - #5351

Open
nlohmann wants to merge 1 commit into
developfrom
claude/todo-206-readme-https
Open

docs: use HTTPS for the astyle and cppcheck links in README#5351
nlohmann wants to merge 1 commit into
developfrom
claude/todo-206-readme-https

Conversation

@nlohmann

@nlohmann nlohmann commented Aug 3, 2026

Copy link
Copy Markdown
Owner

Summary

Two links in the README were still plain http://:

  • astyle.sourceforge.net — serves HTTPS directly, so it just becomes https://.
  • cppcheck.sourceforge.nethttps://cppcheck.sourceforge.net 301-redirects to https://cppcheck.sourceforge.io, which is also the URL already used in docs/mkdocs/docs/community/quality_assurance.md. Linking there directly skips the redirect and keeps the two files consistent.

Both targets were checked and return HTTP 200 over HTTPS.

This redoes #5280, which no longer exists (the author's account appears to have been removed).

Breaking changes to the public API

None — documentation only, no code or headers touched.


This pull request was prepared by Claude Code.

Both links were still `http://`. `astyle.sourceforge.net` serves HTTPS
directly; `cppcheck.sourceforge.net` redirects to
`https://cppcheck.sourceforge.io`, which is also the URL already used in
`docs/mkdocs/docs/community/quality_assurance.md`, so the redirect is
skipped here.

Signed-off-by: Niels Lohmann <mail@nlohmann.me>
@nlohmann nlohmann added this to the Release 3.13.0 milestone Aug 3, 2026
@nlohmann nlohmann added the 🚀 ready to merge Ready to merge - just waiting for CI to complete. label Aug 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation 🚀 ready to merge Ready to merge - just waiting for CI to complete. S

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants