From 6fee7336e4bb6542472764bc349e17ae49990e84 Mon Sep 17 00:00:00 2001 From: Milan Bouchet-Valat Date: Mon, 16 May 2022 10:00:53 +0200 Subject: [PATCH] Recommend defining `materializer(::Type{<:MyType})` For parametric types, defining `materializer(::Type{MyType})` doesn't work, and this can be hard to catch. --- src/Tables.jl | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/Tables.jl b/src/Tables.jl index d3e0328..6f39462 100644 --- a/src/Tables.jl +++ b/src/Tables.jl @@ -368,7 +368,7 @@ default materializer is `Tables.columntable`, which converts any table input int of `Vector`s. It is recommended that for users implementing `MyType`, they define only -`materializer(::Type{MyType})`. `materializer(::MyType)` will then automatically delegate to +`materializer(::Type{<:MyType})`. `materializer(::MyType)` will then automatically delegate to this method. """ function materializer end