Nordstrom / serverless-artillery / 199
42%
master: 99%

LAST BUILD BRANCH: fix-service-name-bug-#96
DEFAULT BRANCH: master
Ran
Jobs 2
Files 309
Run time 12min
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst
199

Pull #41

travis-ci

Erik Erikson 
Add -o/out, Use 'script.yml', Cleanup

0. Add `-o` (`-out`) flag allowing people to declare where to write the output for the `script` command (Fix #31)
1. By default, write out a generated script to `script.yml` unless otherwise configured. (Fix #40)
2. Improve error handling and use a promise pattern for error notification and messaging.
3. Remove the outdated `commander` dependency
4. Add `shortid` for use in creating (very likely to be) unique service names. (Fix #39)
5. Simplify and clarify the command implementations. (Fix #10 and probably other issues)
6. Improve messages by adding empty lines and tabs around the messaging.  Improve a few messages.
7. Delete the no longer used script.backup.yml
8. Correct getSettings method documentation
9. Remove NTP server queries & package.json dependency declaration
10. Add "description", "repository", and "license" in order to avoid warning messages while running `npm install`
11. Clean up the global `script.yml`, adding comments and guidance to help users

Open questions:
1. Should we (during a `configure`) copy the globally stored `node_modules` folder or, alternatively, re-run `npm install`?
2. I get the following error when deploying a local service following a `configure`: { [ServerlessError: Missing required key 'Bucket' in params] name: 'ServerlessError', statusCode: undefined }
Pull Request #41: Add -o/out, Use 'script.yml', Cleanup

8074 of 19414 relevant lines covered (41.59%)

0.83 hits per line

Jobs
ID Job ID Ran Files Coverage
2 199.2 (NPM_VERSION=3) 309
41.59
 Travis Job 199.2
3 199.3 (NPM_VERSION=2) 309
41.59
 Travis Job 199.3
Source Files on build 199