Skip to content

Broken internal link on Gemini Thinking Config blog page#2684

Merged
marklysze merged 4 commits intomainfrom
elva/fix-72444f36
Apr 20, 2026
Merged

Broken internal link on Gemini Thinking Config blog page#2684
marklysze merged 4 commits intomainfrom
elva/fix-72444f36

Conversation

@VasiliyRad
Copy link
Copy Markdown
Contributor

Files changed:

  • website/docs/_blogs/2025-12-29-AG2-Gemini-Thinking-Config-Support/index.mdx

What I checked before submitting:

Fix is correct and verified. The broken link https://docs.ag2.ai/latest/docs/user-guide/llm-config (404) has been replaced with https://docs.ag2.ai/latest/docs/user-guide/basic-concepts/llm-configuration, which resolves to HTTP 200 (with a trailing-slash redirect, as expected). The change is surgical — only the one broken URL is updated, the link text ("AG2 LLM Configuration Guide") is unchanged, and the surrounding markdown formatting is untouched. No regressions introduced. Worth noting: if the old llm-config URL appears in other files across the repo, those would also need updating — but that's a broader sweep and not a blocker for this fix.


Like a river carves its path — small, steady changes shape the landscape. Fixed by Elva.

… page

Internal link to llm-config user guide returns 404.

Bug: file-72444f36
Fixed by Elva.
@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Apr 15, 2026

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
2 out of 3 committers have signed the CLA.

✅ VasiliyRad
✅ marklysze
❌ Elva


Elva seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

@github-actions github-actions Bot added the documentation Improvements or additions to documentation label Apr 15, 2026
@Lancetnik Lancetnik enabled auto-merge April 15, 2026 18:57
@Lancetnik Lancetnik disabled auto-merge April 15, 2026 19:00
@VasiliyRad VasiliyRad enabled auto-merge April 15, 2026 22:15
Copy link
Copy Markdown
Collaborator

@marklysze marklysze left a comment

Choose a reason for hiding this comment

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

Thanks @VasiliyRad, this is a better change that the previously approved one (#2683)

@marklysze marklysze disabled auto-merge April 20, 2026 01:22
@marklysze marklysze merged commit 143e263 into main Apr 20, 2026
22 of 23 checks passed
@marklysze marklysze deleted the elva/fix-72444f36 branch April 20, 2026 01:22
@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 20, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

❗ There is a different number of reports uploaded between BASE (791b3ee) and HEAD (54d9b11). Click for more details.

HEAD has 19 uploads less than BASE
Flag BASE (791b3ee) HEAD (54d9b11)
3.13 3 0
macos-latest 1 0
beta 7 0
ubuntu-latest 5 1
3.14 1 0
3.10 1 0
3.12 1 0
windows-latest 1 0

see 356 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants