You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Any reason for using options / getOption to store the global state and not a package environment? I feel like the latter is more typical and is generally easier to work with.
This is probably just a stylistic difference, though I feel that the semantics around the two differ (options users are allowed to poke with, environments are package internals).
Again, happy to work up a PR with this, as it's a really straightforward change. There would be no user-visible differences.
The text was updated successfully, but these errors were encountered:
I think I'd prefer an environment. I can contribute the change if you're happy with it being changed, otherwise if you'd rather keep it as done via options, just close this.
Any reason for using
options
/getOption
to store the global state and not a package environment? I feel like the latter is more typical and is generally easier to work with.This is probably just a stylistic difference, though I feel that the semantics around the two differ (options users are allowed to poke with, environments are package internals).
Again, happy to work up a PR with this, as it's a really straightforward change. There would be no user-visible differences.
The text was updated successfully, but these errors were encountered: