Skip to content

Commit

Permalink
cosmetic s3 change, add coveralls
Browse files Browse the repository at this point in the history
  • Loading branch information
Jeff Escalante committed Jun 25, 2014
1 parent c5f67eb commit 5826fdd
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 2 deletions.
2 changes: 2 additions & 0 deletions .travis.yml
Original file line number Diff line number Diff line change
@@ -1,5 +1,7 @@
language: node_js
node_js:
- 0.10
after_script:
- npm run coveralls
env:
- secure: "T6+abdXNKRO6tR+JfevVEl1Wc8YN8oOwFQLQQmZWdudkpOnVnwCVMSbk5iyRCVzasYw5ZIiFXtGkeylTi7GmfsB2UI06eMTnC8O3j2+her6ESZdO4Ev3vE8rXG7ifUue8xzly13jXUvCNPUy5vzqSAXdlCLpOaUOQcfGgtOXiGo="
2 changes: 1 addition & 1 deletion lib/deployers/s3/index.coffee
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,7 @@ module.exports = (root, config) ->
return d.promise

module.exports.config =
required: ['secret_key', 'access_key']
required: ['access_key', 'secret_key']
optional: ['region', 'bucket', 'ignore']

errors =
Expand Down
2 changes: 1 addition & 1 deletion readme.md
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
Ship
----

[![npm](http://img.shields.io/npm/v/ship.svg?style=flat)](https://badge.fury.io/js/ship) [![tests](http://img.shields.io/travis/carrot/ship/master.svg?style=flat)](https://travis-ci.org/carrot/ship) [![dependencies](http://img.shields.io/gemnasium/carrot/ship.svg?style=flat)](https://gemnasium.com/carrot/ship)
[![npm](http://img.shields.io/npm/v/ship.svg?style=flat)](https://badge.fury.io/js/ship) [![tests](http://img.shields.io/travis/carrot/ship/master.svg?style=flat)](https://travis-ci.org/carrot/ship) [![coverage](http://img.shields.io/coveralls/carrot/charge.svg?style=flat)](https://coveralls.io/r/carrot/charge) [![dependencies](http://img.shields.io/gemnasium/carrot/ship.svg?style=flat)](https://gemnasium.com/carrot/ship)

This comment has been minimized.

Copy link
@kylemac

kylemac Jun 25, 2014

looks like the coveralls is pointing to charge

This comment has been minimized.

Copy link
@jescalan

jescalan Jun 25, 2014

hah oops! correcting


Multi-platform deployment with node.

Expand Down

0 comments on commit 5826fdd

Please sign in to comment.