Default noDeploy to an empty list

[jpadilla]

Closes #304

Based on #304 (comment) I changed the default for noDeploy to an empty array. Not entirely sure if it might have been enough to remove boto3 and botocore instead.

[dschep]

LGTM, yes removing all noDeploy defaults is correct.

Two things to do still:

• There's still a test failure
• update the docs

[jpadilla]

@dschep any help pointing me at how to resolve the issue here? https://ci.appveyor.com/project/dschep/serverless-python-requirements/builds/21861593

[dschep]

Could you try adding - cmd: pip install -U pip before this line https://github.com/UnitedIncome/serverless-python-requirements/blob/master/appveyor.yml#L3

[jpadilla]

@dschep thanks! I made tests pass on appveyor, but obviously broken now on circleci...

[jdufresne]

Hi @jpadilla @dschep

I'm interested in seeing this improvement land. Let me know if there is anything I can do to lend a helping hand.

[jdufresne]

The "Omitting Packages" section in README will need some editing to make it accurate.

[jpadilla]

@jdufresne just updated this PR with the latest changes in master. Tests are failing again, so I'll need to take a look. Any help is always appreciated.

[jdufresne]

The section in README starting with "To include the default omitted packages ..." should now be removed as well.

[jdufresne]

Suggested doc fix:

diff --git a/README.md b/README.md
index dba4f2f..b1f0054 100644
--- a/README.md
+++ b/README.md
@@ -217,14 +217,6 @@ custom:
       - pytest
 ```

-To include the default omitted packages, set the `noDeploy` option to an empty
-list:
-```yaml
-custom:
-  pythonRequirements:
-    noDeploy: []
-```
-
 ## Extra Config Options

[jdufresne]

lgtm

[jpadilla]

Appveyor seems to still be failing but seems like for some other reason: https://ci.appveyor.com/project/dschep/serverless-python-requirements/builds/24744154#L186

[dschep]

lgtm. and your right appveyor isn't related to this

[jpadilla]

@dschep @jdufresne 🙌 thanks!

[stevenolen]

👋 any chance a release could be cut to include this? It's pretty clunky to have to continue adding a few (soon-to-be) meaningless lines to every project's serverless.yml.