Merge pull request #111 from RelationalAI/axb-excluding-test-from-safe-logging-checks

Excluding test from safe logging checks

Author: bergel

src/linting/extended_checks.jl

@@ -564,7 +564,12 @@ function all_arguments_are_safe(x::EXPR)
     return true
 end
 
-function check(t::LogStatementsMustBeSafe, x::EXPR)
+function check(t::LogStatementsMustBeSafe, x::EXPR, markers::Dict{Symbol,String})
+    if haskey(markers, :filename)
+        # Must have "src/" in the file name, else we just
+        contains(markers[:filename], "test/") && return
+    end
+
     error_msg = "Unsafe logging statement. You must enclose variables and strings with `@safe(...)`."
 
     # @info and its friends