search

jonthegeek / rapid

R 'API' documents
https://rapid.api2r.org/
Other
6 stars 1 forks source link

Rename everything to class_*? #61

Open jonthegeek opened 11 months ago

jonthegeek commented 11 months ago

Eventually this package will (I think?) have functions for accessing, using, and manipulating rapid objects. Should I rename the constructors to (for example) class_rapid() (etc)? If so, should the class remain the simpler name (probably), in which case they'd look something like this:

class_rapid <- S7::new_class(
  "rapid",
  package = "rapid",
  properties = list(
    info = info,
    servers = servers,
    components = component_collection,
    security = security_requirements
  ) ...

This will help separate property names from class constructor names, which avoids https://github.com/RConsortium/S7/issues/373