Try to improve tidy errors on TODOs

WaffleLapkin · WaffleLapkin · commit b311ec02cc4e · 2023-04-26T14:06:36.000Z
diff --git a/src/tools/tidy/src/style.rs b/src/tools/tidy/src/style.rs
@@ -385,10 +385,12 @@ pub fn check(path: &Path, bad: &mut bool) {
             }
             if filename != "style.rs" {
                 if trimmed.contains("TODO") {
-                    err("TODO is deprecated; use FIXME")
+                    err(
+                        "TODO is used for tasks that should be done before merging a PR; If you want to leave a message in the codebase use FIXME",
+                    )
                 }
                 if trimmed.contains("//") && trimmed.contains(" XXX") {
-                    err("XXX is deprecated; use FIXME")
+                    err("Instead of XXX use FIXME")
                 }
                 if any_problematic_line {
                     for s in problematic_consts_strings.iter() {

Original file line number	Diff line number	Diff line change
`@@ -385,10 +385,12 @@ pub fn check(path: &Path, bad: &mut bool) {`
`385`	`385`	`}`
`386`	`386`	`if filename != "style.rs" {`
`387`	`387`	`if trimmed.contains("TODO") {`
`388`		`- err("TODO is deprecated; use FIXME")`
	`388`	`+ err(`
	`389`	`+ "TODO is used for tasks that should be done before merging a PR; If you want to leave a message in the codebase use FIXME",`
	`390`	`+ )`
`389`	`391`	`}`
`390`	`392`	`if trimmed.contains("//") && trimmed.contains(" XXX") {`
`391`		`- err("XXX is deprecated; use FIXME")`
	`393`	`+ err("Instead of XXX use FIXME")`
`392`	`394`	`}`
`393`	`395`	`if any_problematic_line {`
`394`	`396`	`for s in problematic_consts_strings.iter() {`