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

GitHub Action to run CI on Windows, Linux, Mac #110

Open
wants to merge 55 commits into
base: main
Choose a base branch
from

Conversation

joshgoebel
Copy link
Contributor

@joshgoebel joshgoebel commented May 18, 2021

Related: #90 #109

This adds a GitHub action that runs on every push and runs the full CI suite for:

  • Mac
  • Linux
  • Windows (x64/Release)

It also uploads build artifacts for every platform. I share it incase it may prove helpful for anyone else doing custom things with the CLI. This also fixes 3 broken tests with the Windows test suite.

  • Is our existing CI not catching the failing tests or is it just not building all platforms, or?

I vaguely remember a conversation with Ruby about Travis CI vs GHA - but if we aren't running the FULL suite against all platforms on Travis CI then perhaps we should at least be filling in the gaps with GitHib actions - since it's free and all? Slow is better than never when it comes to CI. (Linux is ridiculously fast at least)

Screen Shot 2021-05-18 at 2 31 29 PM

It seems right now that our CI pipeline is broken if PRs can get merged into main while the test suite is still red. This is my solution to fix that. It's also possible we have a visibility problem, hence #90.


  • If we only want the test fixes I'm happy to strip this PR down to just that as well.
  • Regarding the mode test... perhaps we should just remove it?

@joshgoebel joshgoebel force-pushed the github_actions branch 2 times, most recently from f17109e to 309e5fe Compare May 18, 2021 18:42
@joshgoebel joshgoebel changed the title GitHub Action to run CI and tests on Windows, Linux, Mac GitHub Action to run CI on Windows, Linux, Mac May 18, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant