gopsutil icon indicating copy to clipboard operation
gopsutil copied to clipboard

Implement better CI checks

Open Lomanic opened this issue 6 years ago • 7 comments

Linked to #733

We currently use CircleCI for gopsutil CI. As far as I can see it doesn't run for our PRs, it looks like we should enable this option which is disabled by default https://circleci.com/docs/2.0/oss/#build-pull-requests-from-forked-repositories

We should maybe also contact CircleCI to have macOS builders (see here) for free.

As I'm not even admin of the github repo I won't be able to do this, could you @shirou?

I'm also thinking about using Gitlab CI as I'm very familiar with it at work (as I administrate our own self-hosted Gitlab and runners), using their hosted instance at gitlab.com. We could even use our own runners on more platforms than what is proposed for free by CircleCI.

Lomanic avatar Aug 01 '19 08:08 Lomanic

sorry for not say that but yesterday I have enabled that setting.

image

or, if you are familiar with Gitlab CI, it is fine to migrate to it for me.

shirou avatar Aug 01 '19 13:08 shirou

Thanks for setting this. We will see next time a PR is created.

We could have both Gitlab and Circle CIs by the way. I'm leaving this issue open for the GitlabCI part.

Lomanic avatar Aug 01 '19 17:08 Lomanic

I confirm CircleCI ran tests in #735, excellent! :clap:.

For the GitlabCI part, I now remember I won't be able to add it if I'm not Owner of the repo unfortunately.

Lomanic avatar Aug 01 '19 19:08 Lomanic

I think these checks are sufficient for now, it already checks that it compiles on all platforms. I will probably implement a go-fmt check though and will close this afterwards.

Lomanic avatar Aug 18 '19 21:08 Lomanic

Test https://staticcheck.io/docs/ also maybe

Lomanic avatar Aug 20 '19 21:08 Lomanic

https://cirrus-ci.org/features/

To support Open Source community Cirrus CI provides Linux, Windows, macOS and FreeBSD services free of charge.

To be tested.

I'm also curious about builds.sr.ht, which has 1st class support for various BSDs https://man.sr.ht/tutorials/builds.sr.ht/github-integration.md

Lomanic avatar Dec 12 '19 19:12 Lomanic

Thank you for the info!

I have tested Github actions which has Linux/Darwin/Windows support on #805 but no luck...

So I have installed cirrus-ci and start building #806 but not successes yet. I will continue on next time...

shirou avatar Dec 21 '19 04:12 shirou