docs: Special-case command line reference conversion - #29439
docs: Special-case command line reference conversion#29439armandomontanez wants to merge 1 commit into
Conversation
5cc56f7 to
f876ac5
Compare
Special-cases the command line reference conversion so the generated MDX is formatted in a usable way. Since the HTML is rather rigid and predictable, this is sufficient for the initial migration away from DevSite. Eventually, the command line reference should be generated externally from e.g. proto so the generation mechanism is not tied to the bazel version.
f876ac5 to
c031e7b
Compare
|
Ping, the command line reference is pretty broken without this. |
|
Update: I've imported this change, and it's currently running through presubmit checks. I had to fix some smaller lint issues, and I removed the changes to src/main/java/com/google/devtools/build/lib/bazel/repository/RepositoryOptions.java since it will make cherry-picking a bit tricky. Can you please send a separate PR for this file? |
|
@armandomontanez I understand the rationale for this PR. What I don't understand is why there appears to be a custom HTML parser in the CLR converter. Could you use lxml instead, or some other off-the-shelf parser? |
|
✅ Bazel docs preview is ready! Preview URL: https://bazel-pr-29439.mintlify.app/ Updated for |
Sent out #30701 |
Special-cases the command line reference conversion so the generated MDX is formatted in a usable way. Since the HTML is rather rigid and predictable, this is sufficient for the initial migration away from DevSite.
Eventually, the command line reference should be generated externally from e.g. proto so the generation mechanism is not tied to the bazel version.
RELNOTES: None