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
Currently every module import will read several environment variables. This is wasteful, and probably also not intended by the user (particularly for R_BOX_PATH).
The behaviour should be changed to only read these environment variables once at package load time, and setting variables/R options.
Note that this will be a breaking change because changing the value of R_BOX_PATH after ‘box’ has been loaded will now be ignored. However, I think this is an acceptable change that should not affect any existing usage.
implement change
remove deprecation warning
add “breaking change” to NEWS.md
The text was updated successfully, but these errors were encountered:
Currently every module import will read several environment variables. This is wasteful, and probably also not intended by the user (particularly for
R_BOX_PATH
).The behaviour should be changed to only read these environment variables once at package load time, and setting variables/R options.
Note that this will be a breaking change because changing the value of
R_BOX_PATH
after ‘box’ has been loaded will now be ignored. However, I think this is an acceptable change that should not affect any existing usage.NEWS.md
The text was updated successfully, but these errors were encountered: