Skip to content

feat: ignore files listed in .mdbookignore during build#3040

Open
jhult wants to merge 4 commits into
rust-lang:masterfrom
jhult:feat/mdbookignore
Open

feat: ignore files listed in .mdbookignore during build#3040
jhult wants to merge 4 commits into
rust-lang:masterfrom
jhult:feat/mdbookignore

Conversation

@jhult
Copy link
Copy Markdown
Contributor

@jhult jhult commented Feb 25, 2026

Rebase/update of #1908 on top of master.

@rustbot rustbot added the S-waiting-on-review Status: waiting on a review label Feb 25, 2026
@jhult
Copy link
Copy Markdown
Contributor Author

jhult commented Mar 2, 2026

@ehuss, I'd appreciate a review when you get a chance.

@jhult jhult force-pushed the feat/mdbookignore branch from cd3130f to a42c81d Compare March 3, 2026 04:13
@rustbot

This comment has been minimized.

@jhult
Copy link
Copy Markdown
Contributor Author

jhult commented Mar 8, 2026

Other related PRs: #2247 and #2991.

@jhult jhult force-pushed the feat/mdbookignore branch from a42c81d to 6059e1b Compare March 16, 2026 19:25
@rustbot

This comment has been minimized.

@jhult jhult force-pushed the feat/mdbookignore branch 2 times, most recently from 553101f to 4bbff36 Compare March 22, 2026 21:46
@rustbot

This comment has been minimized.

jhult added 4 commits April 21, 2026 19:24
Load .mdbookignore file from src directory and use it to filter files copied during the HTML build process.
Add a new function that uses the  crate to filter files during copy operations, supporting gitignore-style patterns.

build: update Cargo.lock for ignore and pathdiff dependency
@jhult jhult force-pushed the feat/mdbookignore branch from 4bbff36 to 9620641 Compare April 22, 2026 00:26
@rustbot
Copy link
Copy Markdown
Collaborator

rustbot commented Apr 22, 2026

This PR was rebased onto a different master commit. Here's a range-diff highlighting what actually changed.

Rebasing is a normal part of keeping PRs up to date, so no action is needed—this note is just to help reviewers.

@jhult
Copy link
Copy Markdown
Contributor Author

jhult commented May 18, 2026

@ehuss - any thoughts on this?

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

Labels

S-waiting-on-review Status: waiting on a review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants