Merge branch 'master' into remove-unnecessary-curly-braces

Author: bsamuel-ui

.github/dependabot.yml

@@ -0,0 +1,12 @@
+version: 2
+updates:
+- package-ecosystem: npm
+  directory: "/"
+  schedule:
+    interval: daily
+    time: "10:00"
+  open-pull-requests-limit: 10
+  ignore:
+  - dependency-name: eslint
+    versions:
+    - "> 7.22.0"

.github/workflows/lint.yml

@@ -4,19 +4,14 @@ on: [push, pull_request]
 
 jobs:
   build:
-    runs-on: ${{ matrix.os }}
-    strategy:
-      fail-fast: false
-      matrix:
-        os: [ubuntu-latest]
-        node-version: [12]
+    runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v2
 
       - name: Set up Node ${{ matrix.node-version }}
         uses: actions/setup-node@v1
         with:
-          node-version: ${{ matrix.node-version }}
+          node-version: 14
 
       - name: Install deps
         run: npm install

.github/workflows/publish.yml

@@ -6,11 +6,13 @@ jobs:
   publish-npm:
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v1
-      - uses: actions/setup-node@v1
+      - uses: actions/checkout@v2
+
+      - uses: actions/setup-node@v2
         with:
-          version: 12
+          version: 14
           registry-url: https://registry.npmjs.org/
+
       - run: npm publish
         env:
           NODE_AUTH_TOKEN: ${{secrets.npm_token}}

.github/workflows/test.yml

@@ -9,8 +9,7 @@ jobs:
       fail-fast: false
       matrix:
         os: [ubuntu-latest, windows-latest, macOS-latest]
-        python-version: [3.6, 2.7]
-        node-version: [12]
+        python-version: [2.7, 3.6]
     steps:
       - uses: actions/checkout@v2
 
@@ -19,10 +18,10 @@ jobs:
         with:
           python-version: ${{ matrix.python-version }}
 
-      - name: Set up Node ${{ matrix.node-version }}
+      - name: Set up Node
         uses: actions/setup-node@v1
         with:
-          node-version: ${{ matrix.node-version }}
+          node-version: 14
 
       - name: Check python version
         run: |

.gitignore

@@ -45,3 +45,35 @@ __pycache__
 
 #NODE STUFF
 package-lock.json
+yarn.lock
+
+# Lockfiles
+*.lock
+
+# Distribution / packaging
+.Python
+env/
+build/
+develop-eggs/
+dist/
+downloads/
+eggs/
+.eggs/
+lib/
+lib64/
+parts/
+sdist/
+var/
+*.egg-info/
+.installed.cfg
+*.egg
+
+# Serverless directories
+.serverless
+.requirements
+.requirements.zip
+unzip_requirements.py
+
+# Project ignores
+puck/
+serverless.yml.bak

.tool-versions

@@ -528,17 +528,17 @@ package:
 
 ## Contributors
 
-- [@dschep](https://github.com/dschep) - Lead developer & maintainer
+- [@dschep](https://github.com/dschep) - Lead developer & original maintainer
 - [@azurelogic](https://github.com/azurelogic) - logging & documentation fixes
 - [@abetomo](https://github.com/abetomo) - style & linting
 - [@angstwad](https://github.com/angstwad) - `deploy --function` support
 - [@mather](https://github.com/mather) - the cache invalidation option
 - [@rmax](https://github.com/rmax) - the extra pip args option
-- [@bsamuel-ui](https://github.com/bsamuel-ui) - Python 3 support
+- [@bsamuel-ui](https://github.com/bsamuel-ui) - Python 3 support, current maintainer
 - [@suxor42](https://github.com/suxor42) - fixing permission issues with Docker on Linux
 - [@mbeltran213](https://github.com/mbeltran213) - fixing docker linux -u option bug
 - [@Tethik](https://github.com/Tethik) - adding usePipenv option
-- [@miketheman](https://github.com/miketheman) - fixing bug with includes when using zip option
+- [@miketheman](https://github.com/miketheman) - fixing bug with includes when using zip option, update eslint,
 - [@wattdave](https://github.com/wattdave) - fixing bug when using `deploymentBucket`
 - [@heri16](https://github.com/heri16) - fixing Docker support in Windows
 - [@ryansb](https://github.com/ryansb) - package individually support
@@ -551,5 +551,12 @@ package:
 - [@alexjurkiewicz](https://github.com/alexjurkiewicz) - [docs about docker workflows](#native-code-dependencies-during-build)
 - [@andrewfarley](https://github.com/andrewfarley) - Implemented download caching and static caching
 - [@bweigel](https://github.com/bweigel) - adding the `slimPatternsAppendDefaults` option & fixing per-function packaging when some functions don't have requirements & Porting tests from bats to js!
-- [@squaresurf](https://github.com/squaresurf) - adding usePoetry option
+- Poetry support
+	- [@squaresurf](https://github.com/squaresurf)
+	- [@drice](https://github.com/drice)
+	- [@ofercaspi](https://github.com/ofercaspi)
+	- [@tpansino](https://github.com/tpansino)
 - [@david-mk-lawrence](https://github.com/david-mk-lawrence) - added Lambda Layer support
+- [@bryantbriggs](https://github.com/bryantbiggs) - Fixing CI/CD
+- [@jacksgt](https://github.com/jacksgt) - Fixing pip issues
+- [@lephuongbg](https://github.com/lephuongbg) - Fixing single function deployment

README.md

@@ -14,7 +14,7 @@ function cleanup() {
   const artifacts = ['.requirements'];
   if (this.options.zip) {
     if (this.serverless.service.package.individually) {
-      this.targetFuncs.forEach(f => {
+      this.targetFuncs.forEach((f) => {
         artifacts.push(path.join(f.module, '.requirements.zip'));
         artifacts.push(path.join(f.module, 'unzip_requirements.py'));
       });
@@ -25,7 +25,7 @@ function cleanup() {
   }
 
   return BbPromise.all(
-    artifacts.map(artifact =>
+    artifacts.map((artifact) =>
       fse.removeAsync(path.join(this.servicePath, artifact))
     )
   );
@@ -47,7 +47,7 @@ function cleanupCache() {
     const promises = [];
     glob
       .sync([path.join(cacheLocation, '*slspyc/')], { mark: true, dot: false })
-      .forEach(file => {
+      .forEach((file) => {
         promises.push(fse.removeAsync(file));
       });
     return BbPromise.all(promises);

example/.gitignore

@@ -81,7 +81,7 @@ function tryBindPath(serverless, bindPath, testFile) {
     `${bindPath}:/test`,
     'alpine',
     'ls',
-    `/test/${testFile}`
+    `/test/${testFile}`,
   ];
   try {
     if (debug) serverless.cli.log(`Trying bindPath ${bindPath} (${options})`);
@@ -167,7 +167,7 @@ function getDockerUid(bindPath) {
     'stat',
     '-c',
     '%u',
-    '/bin/sh'
+    '/bin/sh',
   ];
   const ps = dockerCommand(options);
   return ps.stdout.trim();

example_native_deps/.gitignore

@@ -21,15 +21,15 @@ function injectRequirements(requirementsPath, packagePath, options) {
 
   return fse
     .readFileAsync(packagePath)
-    .then(buffer => JSZip.loadAsync(buffer))
-    .then(zip =>
+    .then((buffer) => JSZip.loadAsync(buffer))
+    .then((zip) =>
       BbPromise.resolve(
         glob.sync([path.join(requirementsPath, '**')], {
           mark: true,
-          dot: true
+          dot: true,
         })
       )
-        .map(file => [file, path.relative(requirementsPath, file)])
+        .map((file) => [file, path.relative(requirementsPath, file)])
         .filter(
           ([file, relativeFile]) =>
             !file.endsWith('/') &&
@@ -42,7 +42,7 @@ function injectRequirements(requirementsPath, packagePath, options) {
         .mapSeries(([file, relativeFile, fileStat]) =>
           zipFile(zip, relativeFile, fse.readFileAsync(file), {
             unixPermissions: fileStat.mode,
-            createFolders: false
+            createFolders: false,
           })
         )
         .then(() => writeZip(zip, packagePath))
@@ -61,16 +61,16 @@ function moveModuleUp(source, target, module) {
 
   return fse
     .readFileAsync(source)
-    .then(buffer => JSZip.loadAsync(buffer))
-    .then(sourceZip =>
+    .then((buffer) => JSZip.loadAsync(buffer))
+    .then((sourceZip) =>
       sourceZip.filter(
-        file =>
+        (file) =>
           file.startsWith(module + '/') ||
           file.startsWith('serverless_sdk/') ||
           file.match(/^s_.*\.py/) !== null
       )
     )
-    .map(srcZipObj =>
+    .map((srcZipObj) =>
       zipFile(
         targetZip,
         srcZipObj.name.startsWith(module + '/')
@@ -96,18 +96,18 @@ function injectAllRequirements(funcArtifact) {
 
   if (this.serverless.service.package.individually) {
     return BbPromise.resolve(this.targetFuncs)
-      .filter(func =>
+      .filter((func) =>
         (func.runtime || this.serverless.service.provider.runtime).match(
           /^python.*/
         )
       )
-      .map(func => {
+      .map((func) => {
         if (!get(func, 'module')) {
           set(func, ['module'], '.');
         }
         return func;
       })
-      .map(func => {
+      .map((func) => {
         if (func.module !== '.') {
           const artifact = func.package ? func.package.artifact : funcArtifact;
           const newArtifact = path.join(
@@ -122,7 +122,7 @@ function injectAllRequirements(funcArtifact) {
           return func;
         }
       })
-      .map(func => {
+      .map((func) => {
         return this.options.zip
           ? func
           : injectRequirements(

lib/clean.js

@@ -36,7 +36,7 @@ function createLayers() {
       }-${this.serverless.providers.aws.getStage()}-python-requirements`,
       description:
         'Python requirements generated by serverless-python-requirements.',
-      compatibleRuntimes: [this.serverless.service.provider.runtime]
+      compatibleRuntimes: [this.serverless.service.provider.runtime],
     },
     this.options.layer
   );
@@ -55,11 +55,9 @@ function layerRequirements() {
 
   this.serverless.cli.log('Packaging Python Requirements Lambda Layer...');
 
-  return BbPromise.bind(this)
-    .then(zipRequirements)
-    .then(createLayers);
+  return BbPromise.bind(this).then(zipRequirements).then(createLayers);
 }
 
 module.exports = {
-  layerRequirements
+  layerRequirements,
 };