-
-
Notifications
You must be signed in to change notification settings - Fork 280
fix(docs): resolve broken anchors and links in versioned docs #759
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
fix(docs): resolve broken anchors and links in versioned docs #759
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you and congratulations 🎉 for opening your very first pull request in keploy
5625675 to
d5ecfce
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey @VishalPainjane , the changes look good overall, but I noticed a few issues while running it. Could you please check this once? Also, please add the necessary screenshots or a short video demonstrating your changes. Thanks!
d301906 to
dad7cdd
Compare
Achanandhi-M
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey @VishalPainjane, I noticed that around 52 files were changed in this PR. I don’t think all of these changes are necessary—could you please review and limit the changes to what’s required? Thanks!
Signed-off-by: Vishal Painjane <[email protected]>
d742565 to
59d8c5f
Compare
Signed-off-by: Vishal Painjane <[email protected]>
|
@Achanandhi-M, please review it now. Thanks |
Achanandhi-M
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey @VishalPainjane , one more change—we’ve added version 4 as well, but the updates aren’t reflected in the version 4 files yet. Could you please take a look and update those files too?
Signed-off-by: Vishal Painjane <[email protected]>
|
Hi @Achanandhi-M, I have updated the PR to include the fixes for Version 4.0.0 as requested. Changes:
I ran |



What has changed?
This PR fixes multiple broken internal links and anchors in the versioned documentation (v2 and v3) that were causing build failures.
Specifically, it addresses:
../../to../) in quickstart guides, platform requirements, and installation docs to correctly point toserver/installation.md.{#capturing-testcases}toserver/installation.mdin both v2.0.0 and v3.0.0 docs to ensure deep linking works reliably.mac-linux.md,windows-wsl.md) to point to the correct installation headers.This PR Resolves #3649 issue in keploy repo
Type of change
How Has This Been Tested?
I ran
npm run buildlocally to verify that all broken link errors are resolved.Build Output:

Checklist: