Fix broken anchor #760
Fix broken anchor #760
Conversation
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
There was a problem hiding this comment.
Pull request overview
This PR addresses broken anchor warnings that appear during npm run build by fixing internal documentation links and updating the deprecated onBrokenMarkdownLinks configuration option.
Changes:
- Replaced
/docs/server/path prefix with/server/in internal links across multiple documentation files - Changed directory-style links to explicit
.mdfile links (e.g.,/server/installation/→/server/installation.md) - Updated
onBrokenMarkdownLinksconfiguration indocusaurus.config.jsto use a nested structure undermarkdown.hooks
Reviewed changes
Copilot reviewed 9 out of 9 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| versioned_docs/version-3.0.0/keploy-explained/windows-wsl.md | Fixed two broken anchor links by updating path prefix and adding .md extension |
| versioned_docs/version-3.0.0/keploy-explained/mac-linux.md | Fixed two broken anchor links by updating path prefix and adding .md extension |
| versioned_docs/version-3.0.0/installation/windows.md | Fixed broken anchor link by updating path prefix and adding .md extension |
| versioned_docs/version-3.0.0/installation/macos.md | Fixed broken anchor link by updating path prefix and adding .md extension |
| versioned_docs/version-3.0.0/installation/linux.md | Fixed broken anchor link by updating path prefix and adding .md extension |
| versioned_docs/version-3.0.0/concepts/platform-requirements.md | Fixed two broken anchor links in table by updating path prefix and adding .md extension |
| versioned_docs/version-2.0.0/keploy-explained/mac-linux.md | Fixed two broken anchor links by updating path prefix and adding .md extension |
| versioned_docs/version-2.0.0/concepts/platform-requirements.md | Fixed broken anchor link in table by updating path prefix and adding .md extension |
| docusaurus.config.js | Updated deprecated onBrokenMarkdownLinks configuration to use nested structure |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
SaketV8
force-pushed
the
fix-broken-anchor
branch
2 times, most recently
from
a48f9a8 to
f1eb9b4
Compare
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 9 out of 9 changed files in this pull request and generated 1 comment.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
docusaurus.config.js
Outdated
| onBrokenMarkdownLinks: 'warn', | ||
| }, | ||
| }, | ||
| // onBrokenMarkdownLinks: "warn", |
There was a problem hiding this comment.
The commented-out code on this line should be removed since it's no longer needed. Keeping commented-out configuration can cause confusion for future maintainers.
| // onBrokenMarkdownLinks: "warn", |
There was a problem hiding this comment.
Acknowledged, this is intentional for now and will be cleaned up later.
SaketV8
force-pushed
the
fix-broken-anchor
branch
from
f1eb9b4 to
ef5baa3
Compare
There was a problem hiding this comment.
Hey @SaketV8, 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!
|
Hi @Achanandhi-M, I have run the command locally and it worked successfully. I also removed some duplicate values from the config file, so it should now work without any errors. I’ve attached the screenshots and recording for the build.. Screenrecordingnpm start(loading gif...)
Screeenshotnpm start
npm run build
|
Achanandhi-M
left a comment
Achanandhi-M
left a comment
There was a problem hiding this comment.
Hey @SaketV8 but stills i am facing the same issue
|
Hi @Achanandhi-M, in your error log its showing Docusaurus version: Even the current main branch of this repository is using Docusaurus version: here's the linkLine 25 in 6621dd4 Could you please get the latest changes for the
|
|
Hey @SaketV8, that’s not the issue. As you suggested, I checked my package.json, and the version is already 3.7.0. Could you please take another look? 
|
|
Hi @Achanandhi-M , could you please reinstall all dependency of this repository. reinstalling the dependency
after that install using |
|
Hey @SaketV8, thanks for the recommendation. There was an issue on my machine related to a dependency, but it’s working fine now. |
There was a problem hiding this comment.
Hey @SaketV8, 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?
|
Hi @Achanandhi-M , Sure, I’ll check this and update the version 4 as well shortly |
Signed-off-by: Saket Maurya <mauryapriyadarshi2004@gmail.com>
…ce with v4 based onBrokenMarkdownLinks Signed-off-by: Saket Maurya <mauryapriyadarshi2004@gmail.com>
Signed-off-by: Saket Maurya <mauryapriyadarshi2004@gmail.com>
SaketV8
force-pushed
the
fix-broken-anchor
branch
from
9ded3e0 to
e8f7317
Compare
|
Hi @Achanandhi-M I have added the latest changes from Screen recording(loading GIF...)
|
|
Hey @SaketV8 but in the files changes i didn't see any changes in the version 4 files?? |
|
Hi @Achanandhi-M , actually I have to rebase my branch so there is no new changes as all changes for version 4 are in main already and I have just rebased my branch on top of the latest main |
2 participants
What has changed?
This PR fixes the multiple broken anchors warning that appears when running
npm run buildThis PR specifically fixes the build warning by replacing
/docs/server/...with/server/...It also resolves issues by replacing
/routewith/route.md(for example,
/server/installation/-->/server/installation.md).This PR Resolves #3649
Type of change
Please delete options that are not relevant.
How Has This Been Tested?
I have run npm run build, and it build successfully without any errors or warning.
Before
After
Checklist: