Idea: Consolidate all CI in GitHub Actions

b-r-u / osmpbf

A Rust library for reading the OpenStreetMap PBF file format (*.osm.pbf).

Apache License 2.0

122 stars 19 forks source link

Idea: Consolidate all CI in GitHub Actions #25

Open nyurik opened 2 years ago

nyurik commented 2 years ago

Would it make sense to consolidate all CI in a single github workflows platform? Seems weird to have multiple near-identical testing infrastructures that require contributors to understand?

b-r-u commented 2 years ago

I would like to keep AppVeyor around. There is already so much functionality concentrated on the GitHub platform. I'm still sad that Travis is not as easily available anymore.

I'm also fine with our GitHub workflow having more (or less, or other) tests/targets than other CI platforms.

nyurik commented 2 years ago

Ok. I will add a few more targets / tests then:

cargo fmt
cargo clippy
a small test matrix for linux targets - 3 should probably be enough -- the latest (1.62), the one before, and the MSRV (i would recommend 1.58+)

b-r-u commented 2 years ago

That would be great!

nyurik commented 2 years ago

note that we may at least reduce the number of AppVeyor targets - simply to speed up the testing -- they are not very fast it seems, and don't run things in parallel. Which ones do you think are important to keep there?