Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove properly tracked config file from .gitignore & add support for skipping of link-checking #2023

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

marxin
Copy link
Contributor

@marxin marxin commented Jul 24, 2024

The file is part of the git history and is a configuration file.

Fixes: #2018

@camelid
Copy link
Member

camelid commented Jul 26, 2024

I think the reason this is ignored is that we often want to change book.toml locally to turn off link-checking. I'm not sure it makes sense to remove it from the gitignore unless there's another way to disable link-checking locally.

@marxin marxin force-pushed the book.toml-remove-from-gitignores branch from af3a314 to 5b5c220 Compare July 27, 2024 07:38
@marxin marxin changed the title Remove properly tracked config file from .gitignore Remove properly tracked config file from .gitignore & add support for skipping of link-checking Jul 27, 2024
@marxin
Copy link
Contributor Author

marxin commented Jul 27, 2024

Sure, that makes sense and I've just added the support. What do you think about it now?

The file is part of the git history and is a configuration file.

Fixes: rust-lang#2018
@marxin marxin force-pushed the book.toml-remove-from-gitignores branch from 5b5c220 to 9667f99 Compare July 27, 2024 12:54
@marxin
Copy link
Contributor Author

marxin commented Aug 14, 2024

@camelid May I please ping this?

@jieyouxu jieyouxu added the S-waiting-on-review Status: this PR is waiting for a reviewer to verify its content label Oct 8, 2024
@jieyouxu jieyouxu added A-meta Area: meta, about rustc or rustc-dev-guide itself C-CI Category: CI for this repo A-linkcheck Area: linkcheck WG-rustc-dev-guide Working group: rustc-dev-guide labels Nov 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-linkcheck Area: linkcheck A-meta Area: meta, about rustc or rustc-dev-guide itself C-CI Category: CI for this repo S-waiting-on-review Status: this PR is waiting for a reviewer to verify its content WG-rustc-dev-guide Working group: rustc-dev-guide
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Why is book.toml included in .gitignore ?
3 participants