-
-
Notifications
You must be signed in to change notification settings - Fork 69
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
Tidying up sample.env #294
Conversation
You are correct... I have WEB2PY_CONFIG set in my config.fish / .bashrc file. So that was masking the problem. |
I think we should document this in the developing.rst file |
Do you understand poetry well enough to say whether running |
Hmmm. Seems like So I guess that means we can just recommend the plugin as an easy way to deal with the |
login shell environment (such as a .bashrc file) But you can also set them in | ||
the ``.env`` file and as long as you have a RUNESTONE_PATH set commands like | ||
``rsmanage`` and ``runestone`` will try to read and use those variables. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm a bit confused by this sentence compared to the disclaimer about rsmanage
at line 232.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
To be clear, the thing I was confused about was at 232 where it says:
"As of this writing (June 2023) rsmanage does not know about the .env
file so you will have to
set them in your login shell if you want to use rsmanage."
I have been adding support for |
Where does it look for the Anyway, it seems to me in the context of the docs for developers I'd assume they'd be running from the repo not from something they installed with |
Yes, Both I also like the idea of |
I think
.env
files don't actually support variable expansion so I fixed that and removed a strayexport
. And I added a comment about runningpoetry self add poetry-dotenv-plugin
which is actually the real point of this PR.I don't know if the default values I gave for the four
DBURL
variables are right but they work for me in both the docker composed server and when building the book in the shell created bypoetry shell
.