diff options
author | Benjamin Coenen <[email protected]> | 2020-04-17 09:34:58 +0100 |
---|---|---|
committer | Benjamin Coenen <[email protected]> | 2020-04-17 09:34:58 +0100 |
commit | 18802ebe41eb88688c6619b3a975b73d2823baea (patch) | |
tree | c88ae8ffd291b6ba409a9ccf3868e499aa162039 /xtask | |
parent | 0a1585075c65dd6049fc11c641309c89210e222d (diff) |
revert tidy tests changes
Signed-off-by: Benjamin Coenen <[email protected]>
Diffstat (limited to 'xtask')
-rw-r--r-- | xtask/tests/tidy-tests/main.rs | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/xtask/tests/tidy-tests/main.rs b/xtask/tests/tidy-tests/main.rs index ead642acc..101ae19bd 100644 --- a/xtask/tests/tidy-tests/main.rs +++ b/xtask/tests/tidy-tests/main.rs | |||
@@ -35,7 +35,7 @@ fn check_todo(path: &Path, text: &str) { | |||
35 | } | 35 | } |
36 | if text.contains("TODO") || text.contains("TOOD") || text.contains("todo!") { | 36 | if text.contains("TODO") || text.contains("TOOD") || text.contains("todo!") { |
37 | panic!( | 37 | panic!( |
38 | "\nTODO markers or todo! macros should not be committed to the master branch,\n\ | 38 | "\nTODO markers should not be committed to the master branch,\n\ |
39 | use FIXME instead\n\ | 39 | use FIXME instead\n\ |
40 | {}\n", | 40 | {}\n", |
41 | path.display(), | 41 | path.display(), |
@@ -47,9 +47,9 @@ fn check_trailing_ws(path: &Path, text: &str) { | |||
47 | if is_exclude_dir(path, &["test_data"]) { | 47 | if is_exclude_dir(path, &["test_data"]) { |
48 | return; | 48 | return; |
49 | } | 49 | } |
50 | for (line_number, line) in text.lines().enumerate() { | 50 | for line in text.lines() { |
51 | if line.chars().last().map(char::is_whitespace) == Some(true) { | 51 | if line.chars().last().map(char::is_whitespace) == Some(true) { |
52 | panic!("Trailing whitespace in {} at line {}", path.display(), line_number) | 52 | panic!("Trailing whitespace in {}", path.display()) |
53 | } | 53 | } |
54 | } | 54 | } |
55 | } | 55 | } |