Skip to content

Commit

Permalink
Clean up use of pip -e and make deps
Browse files Browse the repository at this point in the history
  • Loading branch information
@tilgovi
tilgovi committed Mar 7, 2015
1 parent 00aab5f commit d2bed93
Showing 1 changed file with 1 addition and 2 deletions.
3 changes: 1 addition & 2 deletions requirements.txt
View file
@@ -1,5 +1,4 @@
-e .
PyYAML
-e .[YAML]
cffi
gunicorn
psycogreen
Expand Down

2 comments on commit d2bed93

@nickstenning
Copy link
Contributor

Choose a reason for hiding this comment

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

I was wondering this before. Why are we installing YAML as an extra? Is there a good reason it's not just in install_requires?

@tilgovi
Copy link
Contributor Author

@tilgovi tilgovi commented on d2bed93 Mar 7, 2015

Choose a reason for hiding this comment

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

Because if you wanted to import parts of h into your application but didn't use the assets.yaml file, either because you're not serving static assets or define them another way, you wouldn't need it.

Please sign in to comment.