Remove /tmp/node-dependencies*

[abetomo]

What did you implement:

Since node-dependencies* will remain in /tmp, cleanup added.

How did you implement it:

Add the flow to be deleted last.

How can we verify it:

File does not remain in /tmp after deploy.

Todos:

• Write tests
• Write documentation
• Fix linting errors
• Make sure code coverage hasn't dropped
• Provide verification config / commands / resources
• Enable "Allow edits from maintainers" for this PR
• Update the messages below

Is this ready for review?: YES
Is it a breaking change?: NO

[horike37]

Hi @abetomo, Thank you for your first-time contribution on this repo 👍
When does this happen? I would like to know how to reproduce it so that I can test and review it properly.

[abetomo]

@horike37 Thank you for review.
As described below, when you deploy, files are created in the /tmp.
Because it is not deleted automatically, the number of files increases.

% ls /tmp/node-dependencies*
zsh: no matches found: /tmp/node-dependencies*
% serverless deploy -s production function -f FUNCTION
...
% ls -lh /tmp/node-dependencies*
-rw-r--r-- 1 vagrant vagrant 25K Aug 14 10:07 /tmp/node-dependencies-76deac41969e904e-dev
-rw-r--r-- 1 vagrant vagrant  38 Aug 14 10:07 /tmp/node-dependencies-76deac41969e904e-prod

% serverless deploy -s production
...
% ls -lh /tmp/node-dependencies*
-rw-r--r-- 1 vagrant vagrant 25K Aug 14 10:19 /tmp/node-dependencies-5a3a322da5f3c0b2-dev
-rw-r--r-- 1 vagrant vagrant  38 Aug 14 10:19 /tmp/node-dependencies-5a3a322da5f3c0b2-prod
-rw-r--r-- 1 vagrant vagrant 25K Aug 14 10:07 /tmp/node-dependencies-76deac41969e904e-dev
-rw-r--r-- 1 vagrant vagrant  38 Aug 14 10:07 /tmp/node-dependencies-76deac41969e904e-prod

[horike37]

Thank you for the response @abetomo 👍
Umm, I tried it again but it was not reproduced.

My guess is this may depends on each enviroments. This PR would not introduce any side effect, so I will merge it once and see how it goes after the next release.

[abetomo]

Thank you.