Copilot commented on code in PR #2042:
URL: https://github.com/apache/apisix-website/pull/2042#discussion_r3232350772
##########
doc/docusaurus.config.js:
##########
@@ -55,7 +55,7 @@ module.exports = {
},
},
},
- onBrokenLinks: 'log',
+ onBrokenLinks: 'warn',
Review Comment:
Switching `onBrokenLinks` to `'warn'` will stop builds from failing, but it
also reduces protection against genuinely broken *internal* docs links (they’ll
no longer fail CI). If the underlying issue is navbar items pointing to routes
that don’t exist in the doc build, a more targeted fix is to define those
navbar entries using `href` (treated as external and typically not checked as a
doc route) instead of `to`, or otherwise scope the link checking so only
cross-workspace routes are excluded. That keeps strict broken-link enforcement
for real docs pages while avoiding false positives.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
