antonbabenko · antonbabenko · Nov 4, 2019 · Nov 2, 2019 · Nov 2, 2019 · Nov 3, 2019
diff --git a/.flake8 b/.flake8
@@ -1,6 +1,5 @@
 [flake8]
-# @todo: Fix code to not ignore these - F405
-ignore = E203, E266, E501, W503, F403, F401, F405
-max-line-length = 79
+ignore = W504, F401
+max-line-length = 150
 max-complexity = 18
 select = B,C,E,F,W,T4,B9
diff --git a/.github/FUNDING.yml b/.github/FUNDING.yml
@@ -1,2 +1,2 @@
 github: [antonbabenko]
-custom: https://www.paypal.me/antonbabenko
+custom: https://www.paypal.me/antonbabenko
diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml
@@ -4,7 +4,7 @@ repos:
   hooks:
     - id: isort
 - repo: git://github.com/pre-commit/pre-commit-hooks
-  rev: v1.4.0
+  rev: v2.4.0
   hooks:
     - id: check-merge-conflict
     - id: requirements-txt-fixer

diff --git a/README.md b/README.md
@@ -77,6 +77,10 @@ $ serverless deploy function --function generate-cloudcraft --stage dev
 $ serverless deploy function --function generate-cloudcraft --stage prod
 ```
 
+## Known issues
+
+* Serverless framework version shouldn't be newer than 1.51.0 unless [this](https://github.com/serverless/serverless/issues/6752) and [this](https://github.com/UnitedIncome/serverless-python-requirements/issues/414) bugs are fixed
+
 ## Authors
 
 This project is created and maintained by [Anton Babenko](https://github.com/antonbabenko) with the help from [different contributors](https://github.com/antonbabenko/modules.tf-lambda/graphs/contributors).

diff --git a/bin/test_lambda_local.sh b/bin/test_lambda_local.sh
diff --git a/handler.py b/handler.py
@@ -6,12 +6,12 @@
 from pprint import pformat, pprint
 
 import requests
-from modulestf.cloudcraft.graph import *
-from modulestf.const import *
-from modulestf.convert import *
+from modulestf.cloudcraft.graph import populate_graph
+from modulestf.const import FINAL_DIR
+from modulestf.convert import convert_graph_to_modulestf_config
 from modulestf.logger import setup_logging
-from modulestf.render import *
-from modulestf.upload import *
+from modulestf.render import prepare_render_dirs, render_from_modulestf_config
+from modulestf.upload import upload_file_to_s3
 
 logger = setup_logging()
 
@@ -41,9 +41,14 @@ def load_data(event):
         logger.info("Blueprint url: %s, response code: %s" % (blueprint_url, r.status_code))
 
         if 403 == r.status_code:
-            raise ValueError("Sharing has been disabled for this blueprint. You have to enable it by clicking 'Export' -> 'Get shareable link' on https://cloudcraft.co/app/", 403)
+            raise ValueError("Sharing has been disabled for this blueprint." +
+                             " You have to enable it by clicking 'Export' -> 'Get shareable link'" +
+                             " on https://cloudcraft.co/app/", 403)
+
         elif r.status_code >= 500:
-            raise ValueError("Something went wrong on cloudcraft side. Can't fetch specified blueprint. Check URL and try again later.", 404)
+            raise ValueError("Something went wrong on cloudcraft side. Can't fetch specified blueprint." +
+                             " Check URL and try again later.", 404)
+
     elif localfile:
         file = open(localfile, 'r')
         data = json.load(file)
@@ -92,7 +97,7 @@ def handler(event, context):
         "body": "",
         "headers": {
             "Location": link,
-            "Access-Control-Allow-Origin" : "*",
+            "Access-Control-Allow-Origin": "*",
             "Access-Control-Allow-Credentials": True
         },
         "statusCode": 302,

diff --git a/input/blueprint_small.json b/input/blueprint_small.json