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

Create prod.txt #172

Closed
wants to merge 2 commits into from
Closed

Conversation

kaysiz
Copy link

@kaysiz kaysiz commented Oct 20, 2017

Added a prod.txt for production requirements

Checklist

  • My branch is up-to-date with the upstream develop branch.
  • I have added necessary documentation (if appropriate).

Which issue does this PR fix?: fixes #
#165
Brief description of what this PR does.
Added an empty prod.txt file to be populated with production requirements
Add issue number here. If you do not solve the issue entirely, please change the message e.g. "First steps for issues #IssueNumber"
First steps for issues #165
Why do we need this PR?:
Other contributers can update the file
If relevant, please include a screenshot.

Demo (optional):

Some tips for you to write the instructions:

  • Prefer bulleted description
  • Start after checking out this branch
  • Include any setup required, such as migrating databases, etc.

Testing instructions:

If there is any work still left to do, please add it here.

TODOs (if any):

A picture of a cute animal (not mandatory but encouraged):

Added a prod.txt for production requirements
@coveralls
Copy link

Coverage Status

Coverage remained the same at 67.619% when pulling 0b1a037 on kaysiz:patch-2 into 575c764 on OpenSourceHelpCommunity:predev.

Added production packages minus coverage and flake
@coveralls
Copy link

Coverage Status

Coverage remained the same at 67.619% when pulling 50c9a4c on kaysiz:patch-2 into 575c764 on OpenSourceHelpCommunity:predev.

@tapaswenipathak tapaswenipathak mentioned this pull request Oct 21, 2017
2 tasks
@@ -0,0 +1,6 @@
dj-database-url==0.4.2
Django==1.11.6
Copy link
Member

Choose a reason for hiding this comment

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

Closing this one, merge this with #171.

@kaysiz kaysiz deleted the patch-2 branch October 21, 2017 01:58
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.

3 participants