Geethapranay1 commented on issue #21747: URL: https://github.com/apache/datafusion/issues/21747#issuecomment-4293192338
Hi @comphead, I have the first version working with lychee in CI. One clarification before I finalize the pr: should the markdown link check validate only local/internal links in tracked markdown files, or should it also validate external HTTP/HTTPS links? Right now I’m leaning toward checking all tracked `*.md` files for local links, while excluding external URLs to avoid flaky CI failures from network issues/rate limits. Rust doc-comment links are covered through the existing `cargo doc` job with `RUSTDOCFLAGS=-D warnings`. Does that scope match what you expected for this issue, or would you prefer external links to be included too? -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
