Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

move CI to Github Actions #2244

Merged
merged 12 commits into from May 21, 2021
Merged

move CI to Github Actions #2244

merged 12 commits into from May 21, 2021

Conversation

flavorjones
Copy link
Member

@flavorjones flavorjones commented May 20, 2021

What problem is this PR intended to solve?

Move to Github Actions in earnest using some of @MSP-Greg's work from #2011 and expanding on it. Related to #2207.

High level goals of this PR:

  • move the majority of CI responsibilities to Github Actions jobs
  • drop the Concourse pipelines
  • keep Appveyor for coverage on 32-bit windows

The checklist

Get Github Actions to CI parity with Concourse:

  • get basic tests running: rubocop
  • get basic tests running: vanilla ubuntu
  • get basic tests running: linux
  • get basic tests running: jruby
  • get basic tests running: valgrind (ci: check how fast/slow valgrind is on GA #2226)
  • get special tests running: musl
  • get special tests running: libxml-ruby
  • get gem tests running: vanilla gem
  • get gem tests running: jruby gem
  • get gem tests running: linux native gem

Additional automation:

@flavorjones flavorjones force-pushed the flavorjones-move-ci-to-actions branch 5 times, most recently from 5353cfd to f86b71b Compare May 20, 2021 03:52
@flavorjones flavorjones force-pushed the flavorjones-move-ci-to-actions branch from a1d05db to e4d2346 Compare May 20, 2021 07:00
Also test-nokogumbo-compatibility.

We pick the highest version because test gem we generate in CI is a
version bump from current, and that's what we want to inspect.

This is necessary because the GA macos image has nokogiri 1.6.8.1
installed into a system dir somewhere. ¯\_(ツ)_/¯
@flavorjones flavorjones force-pushed the flavorjones-move-ci-to-actions branch from e4d2346 to fb7046b Compare May 20, 2021 15:06
@flavorjones flavorjones force-pushed the flavorjones-move-ci-to-actions branch from fb7046b to 5fef4d0 Compare May 20, 2021 18:10
@flavorjones flavorjones force-pushed the flavorjones-move-ci-to-actions branch from b83dffe to 6479c4e Compare May 20, 2021 19:48
Also: add vanilla (source) gem coverage for all supported
rubies plus head; and add jruby-head coverage.
@flavorjones flavorjones force-pushed the flavorjones-move-ci-to-actions branch from 7589930 to 0098565 Compare May 20, 2021 20:17
@flavorjones flavorjones mentioned this pull request May 21, 2021
9 tasks
@flavorjones
Copy link
Member Author

Calling this PR "done", and putting the remaining long tail into #2247.

@flavorjones flavorjones marked this pull request as ready for review May 21, 2021 11:17
@flavorjones flavorjones merged commit 9c6b2bb into main May 21, 2021
@flavorjones flavorjones deleted the flavorjones-move-ci-to-actions branch May 21, 2021 11:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant