JuliaCI / Coverage.jl

Take Julia code coverage and memory allocation results, do useful things with them
MIT License
174 stars 68 forks source link

Test Travis on Windows/Mac/Linux, test on more Julia versions, drop support for Julia 0.7 #270

Closed omus closed 4 years ago

omus commented 4 years ago

Testing on Travis CI only instead of both Appveyor and Travis. Should make maintaining simpler and should allow the CI to finish faster too.

Also, now we are testing on Julia 0.7 and 1.4 which this package supports. Probably we should drop 0.7 and remove it from the Project.toml.

DilumAluthge commented 4 years ago

Yeah drop 0.7.

DilumAluthge commented 4 years ago

@omus Want to squash and rebase?

omus commented 4 years ago

Want to squash and rebase?

I was planning on using the GitHub "Squash and merge"

DilumAluthge commented 4 years ago

@omus Let us allow test failures on nightly for now? And then we can disallow failures on nightly once tests are passing on nightly.

omus commented 4 years ago

We'll need to remove Appveyor once this is merged.

omus commented 4 years ago

I'll wait until tomorrow to merge so I have availability to sort out the loose ends

omus commented 4 years ago

@DilumAluthge I'm not sure if you have admin access to the JuliaCI appveyor group but if you do could you go to: https://ci.appveyor.com/project/JuliaCI/coverage-jl/settings and make sure:

Skip branches without appveyor.yml

is checked?

omus commented 4 years ago

I think we also need to check:

Do not build on "Pull request" events

We may not need this setting

DilumAluthge commented 4 years ago

I don't have admin access. @StefanKarpinski

ararslan commented 4 years ago

Done. I can just delete the AppVeyor project if it's no longer used.

omus commented 4 years ago

We may want to leave it around for a little while in case we want to re-enable it for a reason I'm currently unaware of.