Continue separating out requirements by purpose.

18F · Apr 1, 2018 · ac66061 · ac66061
1 parent a5b3157
commit ac66061
Show file tree

Hide file tree

Showing 5 changed files with 24 additions and 34 deletions.
diff --git a/lambda/requirements-lambda.txt b/lambda/requirements-lambda.txt
@@ -0,0 +1,6 @@
+# These are the only dependencies which must get packaged
+# into every (Python-based) Lambda container build.
+
+strict-rfc3339
+publicsuffix
+
diff --git a/requirements-dev.txt b/requirements-dev.txt
@@ -0,0 +1,5 @@
+# These requirements are useful or needed when developing
+# domain-scan itself.
+
+ipython
+pytest
diff --git a/requirements-gatherers.txt b/requirements-gatherers.txt
@@ -0,0 +1,3 @@
+# censys
+google-cloud-bigquery
+google-auth-oauthlib
diff --git a/requirements-scanners.txt b/requirements-scanners.txt
@@ -1,8 +1,3 @@
-############
-# Scanners
-############
-
-cryptography
 
 # a11y
 pyyaml
@@ -16,3 +11,4 @@ git+https://github.com/dhs-ncats/trustymail.git#egg=trustymail
 
 # sslyze
 sslyze>=1.3.4,<1.4.0
+cryptography
diff --git a/requirements.txt b/requirements.txt
@@ -1,33 +1,13 @@
-##########################
-# Common (Local + Lambda)
-##########################
-#
-# These are the only dependencies which must get packaged
-# into every (Python-based) Lambda container build.
-
-strict-rfc3339
-publicsuffix
 
+########
+# NOTE: Any requirements which are needed for Python-based Lambda function
+# packaging should also be listed in lambda/requirements-lambda.txt.
+# Those are in the bottom section below.
 
-#################
-# Common (Local)
-#################
-
+# Used by parts of domain-scan, but not used inside of Lambda function
+# invocation.
 boto3
-ipython
-
-
-############
-# Gatherers
-############
 
-# censys
-google-cloud-bigquery
-google-auth-oauthlib
-
-
-############
-# Tests
-############
-
-pytest
+# Used in Lanbda functions. Also copied to lambda/requirements-lambda.txt.
+strict-rfc3339
+publicsuffix