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.

Sign up for GitHub

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

Jump to bottom

silence warning when reading .env file that is not necessary when running locally in RStudio #113

Closed
cjyetman opened this issue Feb 9, 2024 · 0 comments · Fixed by #123
Closed

silence warning when reading .env file that is not necessary when running locally in RStudio #113

cjyetman opened this issue Feb 9, 2024 · 0 comments · Fixed by #123

Comments

@cjyetman
Copy link
Member

cjyetman commented Feb 9, 2024 •
edited
Loading

workflow.data.preparation/run_pacta_data_preparation.R

Line 22 in 44758b9

readRenviron(".env")

Reading in the .env is not necessary when running in the local RStudio context/method, so we should silence the warning thrown by this command when the .env is not found when in an interactive session.

It might be ideal to simply skip reading in the .env file when running interactive to avoid a local clone with an .env from running into conflicts with the .env overriding the R_CONFIG_ACTIVE value set by using Sys.setenv(R_CONFIG_ACTIVE = "desktop") as in the recommended local/RStudio method.
@cjyetman cjyetman mentioned this issue Feb 11, 2024
Merged
cjyetman added a commit that referenced this issue Feb 12, 2024
@cjyetman
skip reading .env when running in desktop mode (#123)
06a80ec 
closes #113 
to prevent `.env` from overriding/conflicting with the desired "desktop"
config
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

skip reading .env when running in desktop mode RMI-PACTA/workflow.data.preparation
1 participant
@cjyetman