Skip to content

[rustdoc] Create output file after we checked that the standalone markdown file is valid - #160576

Merged
rust-bors[bot] merged 2 commits into
rust-lang:mainfrom
GuillaumeGomez:rustdoc-standalone-md
Aug 6, 2026
Merged

[rustdoc] Create output file after we checked that the standalone markdown file is valid#160576
rust-bors[bot] merged 2 commits into
rust-lang:mainfrom
GuillaumeGomez:rustdoc-standalone-md

Conversation

@GuillaumeGomez

Copy link
Copy Markdown
Member

This PR makes the creation of the output (HTML) file after we checked that the input markdown is valid to prevent the output file content to be truncated (File::create doc) in any case.

r? @Urgau

@rustbot rustbot added A-run-make Area: port run-make Makefiles to rmake.rs S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. T-rustdoc Relevant to the rustdoc team, which will review and decide on the PR/issue. labels Aug 5, 2026

@Urgau Urgau left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, r=me when CI is green

View changes since this review

Comment thread tests/run-make/rustdoc-markdown-without-title/rmake.rs Outdated
@GuillaumeGomez
GuillaumeGomez force-pushed the rustdoc-standalone-md branch from f5e287f to 0f71ea5 Compare August 5, 2026 15:45
Comment thread tests/run-make/rustdoc/markdown-without-title/rmake.rs
@GuillaumeGomez
GuillaumeGomez force-pushed the rustdoc-standalone-md branch from 0f71ea5 to 1de2daa Compare August 5, 2026 17:18
@GuillaumeGomez

Copy link
Copy Markdown
Member Author

@bors r=Urgau,fmease rollup

@rust-bors

rust-bors Bot commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

📌 Commit 1de2daa has been approved by Urgau,fmease

It is now in the queue for this repository.

@rust-bors rust-bors Bot added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Aug 5, 2026
rust-bors Bot pushed a commit that referenced this pull request Aug 5, 2026
…uwer

Rollup of 12 pull requests

Successful merges:

 - #159014 ([rustdoc] Do not take `doc(cfg())` into account when filtering doctests)
 - #159029 (rustc_llvm: Emit module summaries when using -Clto=fat)
 - #160574 (Update rustc-perf submodule)
 - #154585 (treat no_mangle_generic_items as hard error instead of lint warning)
 - #159473 (Fix, simplify, and document doc meta finalize mode)
 - #157489 (Add CoerceShared field-wise reborrow WF checks)
 - #160532 (Add Enzyme bugfix to support rust+llvm23)
 - #160545 (Reorder the methods in `#[rustc_must_implement_one_of]`)
 - #160558 (Revert "codegen_ssa: no dbginfo for scalable vec local w/ `-O0`")
 - #160566 (Re-enable bool indexing assembly test for LLVM 23)
 - #160569 (Remove `OnDuplicate::Custom`)
 - #160576 ([rustdoc] Create output file after we checked that the standalone markdown file is valid)
@rust-bors
rust-bors Bot merged commit 24a93e7 into rust-lang:main Aug 6, 2026
13 checks passed
@rustbot rustbot added this to the 1.99.0 milestone Aug 6, 2026
rust-timer added a commit that referenced this pull request Aug 6, 2026
Rollup merge of #160576 - GuillaumeGomez:rustdoc-standalone-md, r=Urgau,fmease

[rustdoc] Create output file after we checked that the standalone markdown file is valid

This PR makes the creation of the output (HTML) file after we checked that the input markdown is valid to prevent the output file content to be truncated ([`File::create` doc](https://doc.rust-lang.org/nightly/std/fs/struct.File.html#method.create)) in any case.

r? @Urgau
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-run-make Area: port run-make Makefiles to rmake.rs S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. T-rustdoc Relevant to the rustdoc team, which will review and decide on the PR/issue.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants