Migrate testing to Cirrus CI #375

vinferrer · 2021-01-04T14:49:23Z

Closes #373
Closes #354
Closes #353

Proposed Changes

Adapts all the tests from Circle CI to Cirrus CI
Implements the matrix configuration to makeenv, unittest and integration tasks to simplify the code.
This CI uses the codecov bash uploader. Reading the docs in codecov I realized the merging of the reports is done automatically, no need for the merge_coverage job.
Adds python 3.8 and 3.9 tests
Cirrus CI has no credits charge for open source projects, this includes windows, macOS and linux jobs.

I added everything I wanted to add to this PR.
[Code or tests only] I wrote/updated the necessary docstrings.
[Code or tests only] I ran and passed tests locally.
[Documentation only] I built the docs locally.
My contribution is harmonious with the rest of the code: I'm not introducing repetitions.
My code respects the adopted style, especially linting conventions.
The title of this PR is explanatory on its own, enough to be understood as part of a changelog.
I added or indicated the right labels.

Please, comment on my PR while it's a draft and give me feedback on the development!

vinferrer · 2021-01-19T14:36:06Z

@smoia this is ready for approval

smoia · 2021-01-19T14:40:03Z

Approved and merged!

smoia · 2021-01-19T14:41:16Z

Please open the same PR for phys2denoise though!
@62442katieb and @rmarkello , would you have anything against adopting CirrusCI for peakdet?

vinferrer · 2021-01-19T14:41:37Z

Okay but let me rest

62442katieb · 2021-01-19T16:49:46Z

Please open the same PR for phys2denoise though!
@62442katieb and @rmarkello , would you have anything against adopting CirrusCI for peakdet?

Nothing at all, go ahead!

rmarkello · 2021-01-19T17:11:05Z

Agreed, that's totally fine 👍💯

smoia · 2021-01-21T15:45:23Z