Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: switch to shelljs-release #63

Merged
merged 1 commit into from
Jun 13, 2016
Merged

chore: switch to shelljs-release #63

merged 1 commit into from
Jun 13, 2016

Conversation

nfischer
Copy link
Member

This uses shelljs-release, which is a shelljs implementation of Levi's release script.

The script isn't likely to change much, but at least this will be slightly more modular in case we ever figure out improvements we can make to the release script. Also, this dogfoods our API, which is a smart idea, and lets us do releases from non-unix dev platforms.

Feel free to use this PR as an opportunity to review shelljs-release itself (it seems to work, since I used it to make its own releases).

@codecov-io
Copy link

codecov-io commented Jun 10, 2016

Current coverage is 90.62%

Merging #63 into master will not change coverage

@@             master        #63   diff @@
==========================================
  Files             4          4          
  Lines            32         32          
  Methods           0          0          
  Messages          0          0          
  Branches          0          0          
==========================================
  Hits             29         29          
  Misses            3          3          
  Partials          0          0          

Powered by Codecov. Last updated by 3cf5ee7...c927140

@ariporad
Copy link
Contributor

LGTM!

@nfischer nfischer merged commit aee7fb1 into master Jun 13, 2016
@nfischer nfischer deleted the chore-release-dep branch June 13, 2016 22:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants