pressless
pressless copied to clipboard
Incorrect use of promises in `setup`
Your code seems to assume that Promise-based code AWS calls will run in sequence, waiting for the previous to complete before continuing. This is not true (without async and then awaiting each one). As you require nodejs 6+, and async/await are in nodejs 8+, you'd have to do a required version bump to fix the Promises to work the way the code appears to assume they work...
How would you like to handle this? As is, the code would have to be restructured fairly severely to handle standard error checking properly, or a version bump - neither is a call I can make even with a GitHub Pull Request (especially restructuring all the Promise-based code :P ).