Bugfix: callables are now escaped. Updated Julia version to 1.10. (#133)

* Bugfix: callables are now escapaed. Updated Julia version to 1.10.

* Update CI.yml

Author: kdpsingh

NEWS.md

@@ -1,5 +1,9 @@
 # TidierData.jl updates
 
+## v0.16.4 - 2025-01-11
+- Bugfix: Only functions in Base, Core, and Statistics are not escaped. All other functions and callables are escaped.
+- Updated minimum Julia version to 1.10
+
 ## v0.16.3 - 2024-12-28
 - Bugfix: `@summary` no longer errors with non-numeric columns. Instead, it only reports non-numeric summary stats on non-numeric columns. Minor changes to summary column names to be snake_case.
 - Bugfix: Reverted a bug introduced in v0.13.4, which escaped all macros. Now, string macros remain escaped (i.e., keeping it possible to work with Unitful units, e.g. `u"psi"`), but other macros are *not* escaped to allow for those macros to refer to column names within arguments.

Project.toml

@@ -1,7 +1,7 @@
 name = "TidierData"
 uuid = "fe2206b3-d496-4ee9-a338-6a095c4ece80"
 authors = ["Karandeep Singh"]
-version = "0.16.3"
+version = "0.16.4"
 
 [deps]
 Chain = "8be319e6-bccf-4806-a6f7-6fae938471bc"
@@ -20,9 +20,9 @@ DataFrames = "1.5"
 MacroTools = "0.5"
 Reexport = "0.2, 1"
 ShiftedArrays = "2"
-Statistics = "1.6"
+Statistics = "1.10"
 StatsBase = "0.34, 1"
-julia = "1.9"
+julia = "1.10"
 
 [extras]
 Test = "8dfed614-e22c-5e08-85e1-65c5234f0b40"

src/parsing.jl

@@ -394,20 +394,12 @@ function parse_escape_function(rhs_expr::Union{Expr,Symbol})
         return x
       elseif fn isa Symbol && hasproperty(Statistics, fn) && typeof(getproperty(Statistics, fn)) <: Function
         return x
-      elseif fn isa Symbol && hasproperty(Base, fn) && typeof(getproperty(Base, fn)) <: Type
-        return x
-      elseif fn isa Symbol && hasproperty(Core, fn) && typeof(getproperty(Core, fn)) <: Type
-        return x
-      elseif fn isa Symbol && hasproperty(Statistics, fn) && typeof(getproperty(Statistics, fn)) <: Type
-        return x
       elseif contains(string(fn), r"[^\W0-9]\w*$") # valid variable name
         return :($(esc(fn))($(args...)))
       else
         return x
       end
     elseif @capture(x, fn_.(args__))
-      # if fn in [:esc :in :∈ :∉ :Ref :Set :Cols :(:) :∘ :across :desc :mean :std :var :median :first :last :minimum :maximum :sum :length :skipmissing :quantile :passmissing :startswith :contains :endswith]
-      #  return x
       if fn in not_escaped[]
         return x
       elseif fn isa Symbol && hasproperty(Base, fn) && typeof(getproperty(Base, fn)) <: Function
@@ -416,12 +408,6 @@ function parse_escape_function(rhs_expr::Union{Expr,Symbol})
         return x
       elseif fn isa Symbol && hasproperty(Statistics, fn) && typeof(getproperty(Statistics, fn)) <: Function
         return x
-      elseif fn isa Symbol && hasproperty(Base, fn) && typeof(getproperty(Base, fn)) <: Type
-        return x
-      elseif fn isa Symbol && hasproperty(Core, fn) && typeof(getproperty(Core, fn)) <: Type
-        return x
-      elseif fn isa Symbol && hasproperty(Statistics, fn) && typeof(getproperty(Statistics, fn)) <: Type
-        return x
       elseif contains(string(fn), r"[^\W0-9]\w*$") # valid variable name
         return :($(esc(fn)).($(args...)))
       else