Skip to content
This repository has been archived by the owner on Jan 10, 2023. It is now read-only.

Support Heroku CI #28

Merged
merged 2 commits into from
Oct 8, 2018
Merged

Support Heroku CI #28

merged 2 commits into from
Oct 8, 2018

Conversation

meganemura
Copy link
Contributor

@meganemura meganemura commented Apr 3, 2018

I think we may be able to support Heroku CI.

refs:
https://docs.codecov.io/reference#upload
codeclimate/test-reporter#305

We can also set 'CODECOV_SLUG' via app.json.

related: #21

@meganemura
Copy link
Contributor Author

@codecov
Copy link

codecov bot commented Apr 3, 2018

Codecov Report

Merging #28 into master will increase coverage by 0.07%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #28      +/-   ##
==========================================
+ Coverage   96.35%   96.42%   +0.07%     
==========================================
  Files           1        1              
  Lines         247      252       +5     
==========================================
+ Hits          238      243       +5     
  Misses          9        9
Impacted Files Coverage Δ
lib/codecov.rb 96.42% <100%> (+0.07%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3609e13...3690fcf. Read the comment docs.

@meganemura
Copy link
Contributor Author

Failing tests is due to #29.

@meganemura
Copy link
Contributor Author

meganemura commented Apr 4, 2018

In addition, Heroku CI has a HEROKU_TEST_RUN_NUMBER numeric number. I think this is better for build param. But this is undocumented.

@pablofullana
Copy link

Any ETA for this one?

@olmar
Copy link

olmar commented May 25, 2018

Any news about this PR?

@ALexALed
Copy link

@meganemura Any news about this?

@meganemura
Copy link
Contributor Author

@stevepeak Would you please review this?

@meganemura
Copy link
Contributor Author

In addition, this PR works well with my projects.

@meganemura
Copy link
Contributor Author

@stevepeak ping

3 similar comments
@meganemura
Copy link
Contributor Author

@stevepeak ping

@meganemura
Copy link
Contributor Author

@stevepeak ping

@meganemura
Copy link
Contributor Author

@stevepeak ping

@meganemura meganemura force-pushed the heroku-ci branch 2 times, most recently from 057e3ed to 8fe0fb1 Compare October 6, 2018 03:11
@meganemura
Copy link
Contributor Author

@hootener Would you please review this?

@hootener hootener self-requested a review October 8, 2018 12:39
@hootener hootener merged commit 984d796 into codecov:master Oct 8, 2018
@hootener
Copy link
Contributor

hootener commented Oct 8, 2018

Merged and deployed to rubygems 👍

@meganemura meganemura deleted the heroku-ci branch October 8, 2018 15:17
@meganemura
Copy link
Contributor Author

Thanks 😃

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

Successfully merging this pull request may close these issues.

5 participants