Skip to content

Broken link to agenteval_cq_math.ipynb#2677

Merged
marklysze merged 1 commit intomainfrom
elva/fix-7a8d1ff7
Apr 20, 2026
Merged

Broken link to agenteval_cq_math.ipynb#2677
marklysze merged 1 commit intomainfrom
elva/fix-7a8d1ff7

Conversation

@VasiliyRad
Copy link
Copy Markdown
Contributor

Files changed:

  • website/docs/_blogs/2023-11-20-AgentEval/index.mdx

What I checked before submitting:

Fix is correct and verified. Both occurrences of the broken notebook URL (agenteval_cq_math.ipynbagentchat_agenteval_cq_math.ipynb) in the blog post have been updated consistently. The replacement URL resolves to HTTP 200 (no redirect). The change is minimal and surgical — 2 lines in 1 file. No regressions introduced.


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

GitHub notebook URL returns HTTP 404.

Bug: file-7a8d1ff7
Fixed by Elva.
@github-actions github-actions Bot added the documentation Improvements or additions to documentation label Apr 15, 2026
@CLAassistant
Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.


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.

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!

@marklysze marklysze merged commit 0c7140c into main Apr 20, 2026
23 of 24 checks passed
@marklysze marklysze deleted the elva/fix-7a8d1ff7 branch April 20, 2026 01:05
@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 20, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
see 25 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.

3 participants