matomo-org / matomo-sdk-android

SDK for Android to measure your apps with Matomo. Works on Android phones, tablets, Fire TV sticks, and more!

BSD 3-Clause "New" or "Revised" License

390 stars 162 forks source link

Github actions #293

Closed hannesa2 closed 3 years ago

hannesa2 commented 4 years ago

advantage:

faster
easy to run Espresso tests, but in this repo there are no Espresso test, so I didn't run it

hannesa2 commented 4 years ago

If you like it, you could remove Travis

d4rken commented 4 years ago

I like it, but I don't want to change it "just because", it would need to provide a specific benefit that we are currently missing.

Currently the CodeCoverage is broken again, would this fix it? Currently don't have time to investigate Travis again :grin:...

hannesa2 commented 4 years ago

I only applied the same commands like in Travis. Broken or not, at least there is no error.

I at least move complete from Travis to Github actions because of the two reasons https://github.com/matomo-org/matomo-sdk-android/pull/293#issue-465407473

codecov-io commented 3 years ago

Codecov Report

Merging #293 (6daacd3) into master (9c3ccd2) will not change coverage. The diff coverage is n/a.

@@      Coverage Diff      @@
##   master   #293   +/-   ##
=============================
=============================

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 9c3ccd2...6daacd3. Read the comment docs.

d4rken commented 3 years ago

Will fix this up on a different branch.