Skip to content

docs: drop dead large-PRs link from usage index - #3342

Merged
IsmaelMartinez merged 1 commit into
The-PR-Agent:mainfrom
utsab345:fix/docs-large-prs-link-3332
Sep 13, 2026
Merged

IsmaelMartinez merged 1 commit into
The-PR-Agent:mainfrom
utsab345:fix/docs-large-prs-link-3332

Conversation

@utsab345

Copy link
Copy Markdown
Contributor

Fixes #3332

Why

docs/docs/usage-guide/index.md:24 links to ./additional_configurations.md#working-with-large-prs, but the "Working with large PRs" section was fully HTML-commented out (additional_configurations.md:122-136), so the anchor resolves to nothing.

Change

Removed the dead TOC entry from index.md. The referenced section deliberately no longer exists (outdated GPT-4/8000-token guidance), so the link is dropped rather than the section being restored. No other files link to that anchor.

Verification

  • test_docs_nav_parity.py and test_config_reference.py pass.
  • Full unit suite passes (7829 passed, 33 skipped, 1 xfailed).

@github-actions github-actions Bot added the documentation Improvements or additions to documentation label Sep 13, 2026
@qodo-free-for-open-source-projects

Copy link
Copy Markdown
Contributor

Code Review by Qodo

🐞 Bugs (0) 📘 Rule violations (0) 📎 Requirement gaps (0)

Grey Divider

Great, no issues found!

Qodo reviewed your code and found no material issues that require review
  • Author self-review: I have reviewed the code review findings, and addressed the relevant ones.

Grey Divider

Tip of the day
💡 Did you know, you can start a comment with 'qodo' or '@qodo' to chat about any finding

More tips ↗ | Customize Qodo ↗ | Qodo docs ↗

Grey Divider

Qodo Logo

@qodo-free-for-open-source-projects

Copy link
Copy Markdown
Contributor

PR Summary by Qodo

Remove dead large-PR guidance link from usage index

🐞 Bug fix 📝 Documentation 🕐 Less than 5 minutes

Grey Divider

AI Description

• Removes the usage-guide link to a nonexistent large-PR section.
• Aligns navigation with the intentional retirement of outdated model guidance.
High-Level Assessment

Removing the dead navigation entry is the optimal approach. Restoring the section was considered but dismissed because its GPT-4 token-limit and model guidance is intentionally retired and outdated.

Files changed (1) +0 / -1

Other (1) +0 / -1
index.mdRemove dead large-PR documentation link +0/-1

Remove dead large-PR documentation link

• Removes the usage-index entry pointing to the intentionally hidden and outdated “Working with large PRs” section.

docs/docs/usage-guide/index.md

@IsmaelMartinez IsmaelMartinez left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for grepping the rest of the docs before choosing removal over restoration.

Verified: the target heading is inside the comment block at additional_configurations.md:122, the slug is absent from the page's heading list, and index.md:24 was the only link to it anywhere under docs/. The commented section is outdated GPT-4 guidance, so dropping the entry is the right fix.

@IsmaelMartinez
IsmaelMartinez merged commit 417b300 into The-PR-Agent:main Sep 13, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation 🕐 Less than 5 minutes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

docs: usage-guide/index.md links to #working-with-large-prs, a section that is fully commented out

2 participants