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

Reenabling Drone #155

Merged
merged 1 commit into from
Feb 5, 2017
Merged

Reenabling Drone #155

merged 1 commit into from
Feb 5, 2017

Conversation

kasperlewau
Copy link
Contributor

With Drone dropping the hosted service (https://archive.drone.io) and going open source (https://github.com/drone/drone), I wanted to give it and Coveralls a shot.

My initial setup consisted of setting the COVERALLS_REPO_TOKEN env variable and running nyc on top of my mocha tests as part of my builds on a self-hosted Drone CI.

export COVERALLS_REPO_TOKEN=...
nyc mocha && nyc report | coveralls

This worked just dandy™ - but there wasn't enough data being sent through the coveralls script to coveralls.io to get the status API / automated comments going.


This PR re-enables Drone as a CI environment and supplies some more of the available Drone environment data (http://readme.drone.io/0.5/usage/environment-reference/).

  • Drone build number
  • PR number
  • Commit author name & email
  • Commit SHA
  • Commit message
  • Git Branch

So far, I've made one successful build with Coveralls submitting a comment as well as using the Github Status API.

screenie

@coveralls
Copy link

coveralls commented Jan 31, 2017

Coverage Status

Coverage increased (+0.1%) to 95.192% when pulling d117e3d on kasperlewau:master into c81c084 on nickmerwin:master.

3 similar comments
@coveralls
Copy link

Coverage Status

Coverage increased (+0.1%) to 95.192% when pulling d117e3d on kasperlewau:master into c81c084 on nickmerwin:master.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.1%) to 95.192% when pulling d117e3d on kasperlewau:master into c81c084 on nickmerwin:master.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.1%) to 95.192% when pulling d117e3d on kasperlewau:master into c81c084 on nickmerwin:master.

@nickmerwin nickmerwin merged commit 3edbecb into nickmerwin:master Feb 5, 2017
@nickmerwin
Copy link
Owner

Thanks @kasperlewau!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants