Skip to content

Commit 60c080c

Browse files
authored
Recommend defining materializer(::Type{<:MyType}) (#282)
For parametric types, defining `materializer(::Type{MyType})` doesn't work, and this can be hard to catch.
1 parent 138c5be commit 60c080c

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

src/Tables.jl

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -368,7 +368,7 @@ default materializer is `Tables.columntable`, which converts any table input int
368368
of `Vector`s.
369369
370370
It is recommended that for users implementing `MyType`, they define only
371-
`materializer(::Type{MyType})`. `materializer(::MyType)` will then automatically delegate to
371+
`materializer(::Type{<:MyType})`. `materializer(::MyType)` will then automatically delegate to
372372
this method.
373373
"""
374374
function materializer end

0 commit comments

Comments
 (0)