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

Improve documentation of requirements and local settings #78

Merged
merged 2 commits into from
Jul 13, 2020

Conversation

ushkarev
Copy link
Contributor

@ushkarev ushkarev commented Jul 6, 2020

No description provided.

… which explains how to have local settings for development that do not get committed
@ushkarev ushkarev requested a review from a team as a code owner July 6, 2020 16:13
README.rst Outdated

Usage
-----

It's recommended that you use a python virtual environment to isolate each application.
Please call this ``venv`` and make sure it's in the root folder of this application so that
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just adding a note the comments on ministryofjustice/money-to-prisoners-api#600 probably apply here too (only if anything changes of course).

@ushkarev ushkarev requested a review from a team July 10, 2020 10:21
@ushkarev ushkarev merged commit 0435fd2 into master Jul 13, 2020
@ushkarev ushkarev deleted the sample-settings branch July 13, 2020 14:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants