skip cleanup of files that don't match the regex

[Matthew-Smith]

We had an issue where we had added a couple files to our bucket which was used by serverless

The error we saw:

Type Error ---------------------------------------------
 
  Cannot read property '4' of null
 
     For debugging logs, run again after setting the "SLS_DEBUG=*" environment variable.
 
  Get Support --------------------------------------------
     Docs:          docs.serverless.com
     Bugs:          github.com/serverless/serverless/issues
     Issues:        forum.serverless.com
 
  Your Environment Information ---------------------------
     OS:                     linux
     Node Version:           10.16.0
     Serverless Version:     1.44.1
 
ERROR: Job failed: command terminated with exit code 1

basically, file.name.match(/(serverless)\/(.+)\/(.+)\/(\d+)-(.+)\/(.+\.zip)/)[4] was failing because foo.js did not match the regex (returns null)

[eLRuLL]

any plans for merging this fix?

I had to add the main.py file directly into the root of the artifact zip. Getting the same error.

[pethersonmoreno]

Suggested change

	const objectsToRemove = _.pullAllWith(files, objectsToKeep, _.isEqual);
	const objectsToRemove = _.pullAllWith(filteredFiles, objectsToKeep, _.isEqual);

You can't just use all files list because there are directory paths that has other files that we want to keep in objectsToKeep, then I suggest this change on the code

[pethersonmoreno]

I found the same error and with the suggestion the problem was fixed to me testing on local.
These changes solves the problem in Issue: #228