search

RConsortium / S7

S7: a new OO system for R
https://rconsortium.github.io/S7
Other
408 stars 37 forks source link

export `class_NULL`? #489

Open t-kalinowski opened 2 weeks ago

t-kalinowski commented 2 weeks ago

Should we add class_NULL? The primary benefit would be improved discoverability through autocompletions on S7::class_* and in the package documentation.

hadley commented 2 weeks ago

As opposed to just telling people to use NULL?

t-kalinowski commented 2 weeks ago

I think the core issue is that S7's support for a union class like NULL | class_double is not very discoverable at the moment. Perhaps a better solution is to add examples in ?new_property, ?new_class, and/or ?method<-.

mmaechler commented 2 weeks ago

I'd also think it's overkill. Knowning about and using NULL should really be part of every intermediate level useR's tools.