Skip to content

Conversation

@e-esakman
Copy link
Contributor

@e-esakman e-esakman commented Nov 1, 2025

Fixes: #4698

Untitled.design.mp4

Fix: Updated outdated installation/setup links in the Contribute page and sidebar to ensure correct redirection and improve contributor onboarding.

Summary by CodeRabbit

  • Documentation
    • Updated documentation and contribution links to point to their new locations, ensuring users are directed to the correct resources.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Nov 1, 2025

Walkthrough

Documentation reference links in HTML templates were updated to point to new locations in the OWASP-BLT repository. Three occurrences in the contribute page were modified from Setup.md to docs/Setup.md, and one sidenav link was updated to reference an external CONTRIBUTING.md file.

Changes

Cohort / File(s) Summary
Template Documentation Link Updates
website/templates/contribute.html, website/templates/includes/sidenav.html
Updated documentation reference URLs in Contribute section and sidenav to point to new docs/Setup.md and external CONTRIBUTING.md locations in OWASP-BLT repository.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

  • These are straightforward URL/link updates with no logic, control flow, or structural changes involved.

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. You can run @coderabbitai generate docstrings to improve docstring coverage.
✅ Passed checks (2 passed)
Check name Status Explanation
Title Check ✅ Passed The pull request title "Fix: update setup links in contribute page & sidebar" clearly and accurately describes the main changes in the changeset. The title directly addresses the primary update—fixing setup links in both the contribute template and sidebar navigation—which aligns with the actual file modifications (contribute.html and sidenav.html). The title is concise at 9 words, uses specific and descriptive language without vague terms, and provides sufficient context for a developer scanning the repository history to understand the purpose of the change at a glance.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-project-automation github-project-automation bot moved this from Backlog to Ready in 📌 OWASP BLT Project Board Nov 1, 2025
@DonnieBLT DonnieBLT merged commit f8a9c97 into OWASP-BLT:main Nov 1, 2025
16 of 17 checks passed
@DonnieBLT DonnieBLT added the tiny label Nov 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

Broken installation/setup links in Contribute page and sidebar

2 participants