skip check of .po agreeing with .mo (#6518)

MichaelChirico · web-flow · commit ec84fb77ef41 · 2024-09-21T08:29:38.000-07:00
* skip check of .po agreeing with .mo

* Actually use sprintf

* oops
diff --git a/.ci/linters/po/msgfmt_linter.R b/.ci/linters/po/msgfmt_linter.R
@@ -20,6 +20,9 @@ msgfmt_linter <- function(po_file) {
     stop(po_file, " has not been compiled as ", mo_ref, ". Please fix.")
   }
   if (tools::md5sum(mo_ref) == tools::md5sum(mo_tmp)) return(invisible())
+  # TODO(#6517): Re-activate this part of the check to ensure .mo is up to date.
+  cat(sprintf("Note: MD5 sum of msgfmt output for %s does not match %s.\n", po_file, mo_ref))
+  return(invisible())
 
   # NB: file.mtime() will probably be wrong, it will reflect the check-out time of the git repo.
   last_edit_time = system2("git",

Original file line number	Diff line number	Diff line change
`@@ -20,6 +20,9 @@ msgfmt_linter <- function(po_file) {`
`20`	`20`	`stop(po_file, " has not been compiled as ", mo_ref, ". Please fix.")`
`21`	`21`	`}`
`22`	`22`	`if (tools::md5sum(mo_ref) == tools::md5sum(mo_tmp)) return(invisible())`
	`23`	`+ # TODO(#6517): Re-activate this part of the check to ensure .mo is up to date.`
	`24`	`+ cat(sprintf("Note: MD5 sum of msgfmt output for %s does not match %s.\n", po_file, mo_ref))`
	`25`	`+ return(invisible())`
`23`	`26`
`24`	`27`	`# NB: file.mtime() will probably be wrong, it will reflect the check-out time of the git repo.`
`25`	`28`	`last_edit_time = system2("git",`