diff options
| author | merge-script <fanquake@gmail.com> | 2024-07-03 11:42:32 +0100 |
|---|---|---|
| committer | merge-script <fanquake@gmail.com> | 2024-07-03 11:42:32 +0100 |
| commit | 3714692644f45808a6480525abc36870aeee1de4 (patch) | |
| tree | 42ef3e367d21bba71a67316e165694e5f63cd0f9 /test/lint/test_runner | |
| parent | 173ab0ccf2164dd8cd3d486559dffda0d4a1a813 (diff) | |
| parent | dea7afd5e4c2fda7def2e06cfc44dbd617d7cdc2 (diff) | |
Merge bitcoin/bitcoin#30380: lint: Ignore files ignored by git in the Markdown Link Checker
dea7afd5e4c2fda7def2e06cfc44dbd617d7cdc2 lint: remove unneeded trailing line fix (willcl-ark)
4d942547a8155cca4fe2f68afccdb822a174ab1a lint: ignore files ignored by git in mlc (willcl-ark)
Pull request description:
Updating to MLC v0.18.0 includes a new feature which will ignore all files ignored by git: `mlc --gitignore`.
This helps avoid false-positives flagged by this linter in non-project files, such as a developer might expect to have in their working directory (e.g. guix-builds, python venvs, etc.)
Top commit has no ACKs.
Tree-SHA512: 1752448e0c85abd3c73570a17cc69294de2248d7773c6499833ae33806f6c03f3f345261aa7b855a557b45982fbdcb8190e758d087c43b4fb0254fbb39173432
Diffstat (limited to 'test/lint/test_runner')
| -rw-r--r-- | test/lint/test_runner/src/main.rs | 8 |
1 files changed, 2 insertions, 6 deletions
diff --git a/test/lint/test_runner/src/main.rs b/test/lint/test_runner/src/main.rs index 9c35898c1f..2ba58a6da2 100644 --- a/test/lint/test_runner/src/main.rs +++ b/test/lint/test_runner/src/main.rs @@ -410,6 +410,7 @@ fn lint_markdown() -> LintResult { "--offline", "--ignore-path", md_ignore_path_str.as_str(), + "--gitignore", "--root-dir", ".", ]) @@ -419,11 +420,6 @@ fn lint_markdown() -> LintResult { Ok(output) if output.status.success() => Ok(()), Ok(output) => { let stderr = String::from_utf8_lossy(&output.stderr); - let filtered_stderr: String = stderr // Filter out this annoying trailing line - .lines() - .filter(|&line| line != "The following links could not be resolved:") - .collect::<Vec<&str>>() - .join("\n"); Err(format!( r#" One or more markdown links are broken. @@ -433,7 +429,7 @@ Relative links are preferred (but not required) as jumping to file works nativel Markdown link errors found: {} "#, - filtered_stderr + stderr )) } Err(e) if e.kind() == ErrorKind::NotFound => { |