chore(python) - Add support for python 3.8/3.9 remove 3.6 (#219)

* Remove python 3.6 and in python 3.8/3.9

Author: kddejong

.github/workflows/pr-ci.yml

@@ -12,13 +12,20 @@ on:
 jobs:
   build:
     env:
-      AWS_DEFAULT_REGION: us-east-1
+      AWS_DEFAULT_REGION: "us-east-1"
+      AWS_SDK_LOAD_CONFIG: 1
+      GOPATH: ${{ github.workspace }}
     runs-on: ubuntu-latest
     strategy:
       matrix:
-        python: [3.6, 3.7, 3.8]
+        python: [3.7, 3.8, 3.9]
+    defaults:
+      run:
+        working-directory: ${{ env.GOPATH }}/src/github.com/${{ github.repository }}
     steps:
-    - uses: actions/checkout@v2
+    - uses: actions/checkout@v3
+      with:
+        path: ${{ env.GOPATH }}/src/github.com/${{ github.repository }}
     - uses: actions/setup-go@v3
       with:
         go-version: '>=1.17.0'
@@ -30,6 +37,7 @@ jobs:
       run: |
         pip install pre-commit
         pip install --upgrade mypy 'attrs==19.2.0' -r https://raw.githubusercontent.com/aws-cloudformation/aws-cloudformation-rpdk/master/requirements.txt
+        go get ./...
         go install github.com/go-critic/go-critic/cmd/gocritic@latest
         go install golang.org/x/tools/cmd/goimports@latest
         go install github.com/fzipp/gocyclo/cmd/gocyclo@latest

cfn/cfn.go

@@ -3,10 +3,8 @@ package cfn
 import (
 	"context"
 	"errors"
-	"io/ioutil"
 	"log"
 	"os"
-	"path"
 	"sync"
 	"time"
 
@@ -103,10 +101,7 @@ func makeEventFunc(h Handler) eventFunc {
 			logging.SetProviderLogOutput(l)
 		})
 		re := newReportErr(m)
-		if err := scrubFiles("/tmp"); err != nil {
-			log.Printf("Error: %v", err)
-			m.PublishExceptionMetric(time.Now(), event.Action, err)
-		}
+
 		handlerFn, cfnErr := router(event.Action, h)
 		log.Printf("Handler received the %s action", event.Action)
 		if cfnErr != nil {
@@ -145,17 +140,6 @@ func makeEventFunc(h Handler) eventFunc {
 	}
 }
 
-func scrubFiles(dir string) error {
-	names, err := ioutil.ReadDir(dir)
-	if err != nil {
-		return err
-	}
-	for _, entery := range names {
-		os.RemoveAll(path.Join([]string{dir, entery.Name()}...))
-	}
-	return nil
-}
-
 // router decides which handler should be invoked based on the action
 // It will return a route or an error depending on the action passed in
 func router(a string, h Handler) (handlerFunc, cfnerr.Error) {

python/rpdk/go/codegen.py

@@ -12,10 +12,8 @@
 from subprocess import PIPE, CalledProcessError, run as subprocess_run  # nosec
 from tempfile import TemporaryFile
 
-from . import __version__
 from .resolver import translate_type
 from .utils import safe_reserved, validate_path
-from .version import check_version
 
 LOG = logging.getLogger(__name__)
 

setup.py

@@ -51,8 +51,9 @@ def find_version(*file_paths):
         "Topic :: Software Development :: Code Generators",
         "Operating System :: OS Independent",
         "Programming Language :: Python :: 3 :: Only",
-        "Programming Language :: Python :: 3.6",
         "Programming Language :: Python :: 3.7",
+        "Programming Language :: Python :: 3.8",
+        "Programming Language :: Python :: 3.9",
     ],
     keywords="Amazon Web Services AWS CloudFormation",
 )