Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Read environment variables only at startup #344

Open
3 tasks
klmr opened this issue Oct 22, 2023 · 0 comments
Open
3 tasks

Read environment variables only at startup #344

klmr opened this issue Oct 22, 2023 · 0 comments
Assignees

Comments

@klmr
Copy link
Owner

klmr commented Oct 22, 2023

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
@klmr klmr self-assigned this Oct 22, 2023
@klmr klmr changed the title Only read environment variables at startup Read environment variables only at startup Oct 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant