-
-
Notifications
You must be signed in to change notification settings - Fork 127
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
Update configuration docs #521
Conversation
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.
Do we even need a docs/
directory at this point, since all of the configuration here is essentially duplicated in the contributing guide?
Yes, but also no. The |
Is The only variables I see that are missing in the contributing guide are |
Partially, yes. |
I'll discuss this with the core devs then get back to you, thanks for the other minor enhancements by the way! |
@jchristgit The consensus is in favour of removing the |
Yes, I'd be happy to do so. I'll try to do that today, if that doesn't work, it'll chime in here next week |
When I started the Django app in January 2019, I documented a setup guide and environment variables for local configuration of the app here. These were lacking the new
$METRICITY_DB_URL
environment variable. Thesetup.md
file was also changed to simply be a symlink to the current contributing guide for the site.