nstrayer / shinysense

A bunch of shiny modules that let shiny sense the world around it.
http://nickstrayer.me/shinysense
Other
206 stars 31 forks source link

Inconsistant naming schemes #23

Closed nstrayer closed 5 years ago

nstrayer commented 6 years ago

For some reason, I chose verbs for some (shinyviewr, and shinydrawr) but then went with nouns for others (shinyearr).

Ultimately I would like to rename the functions to simple verbs of their actions, taking away the shiny- prefix as I think it's superfluous as it can be registered with something like shinysense::viewr...

nstrayer commented 5 years ago

Fixed this. I ended up choosing to keep shiny prefix for all the functions that were meant for a shiny app, and simply omit it for the ones that work outside of a shiny app. Changes finalized in https://github.com/nstrayer/shinysense/pull/39.