Skip to content

Run CI against Mac#63

Merged
cburgdorf merged 1 commit intoargotorg:masterfrom
cburgdorf:christoph/ci/mac-win
Oct 2, 2020
Merged

Run CI against Mac#63
cburgdorf merged 1 commit intoargotorg:masterfrom
cburgdorf:christoph/ci/mac-win

Conversation

@cburgdorf
Copy link
Collaborator

@cburgdorf cburgdorf commented Oct 1, 2020

What was wrong?

We only test against Linux

How was it fixed?

Added support for Mac

To-Do

  • OPTIONAL: Update Spec if applicable
  • Clean up commit history

@cburgdorf cburgdorf force-pushed the christoph/ci/mac-win branch from abe0180 to 29a7617 Compare October 1, 2020 09:38
@codecov-commenter
Copy link

codecov-commenter commented Oct 1, 2020

Codecov Report

Merging #63 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #63   +/-   ##
=======================================
  Coverage   70.86%   70.86%           
=======================================
  Files          29       29           
  Lines        2073     2073           
=======================================
  Hits         1469     1469           
  Misses        604      604           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 503d7bb...e5c9221. Read the comment docs.

@cburgdorf cburgdorf force-pushed the christoph/ci/mac-win branch 10 times, most recently from 8aa71b5 to 2f66692 Compare October 1, 2020 16:18
@cburgdorf
Copy link
Collaborator Author

Ok, finally hitting the point where a cache entry was restored

image

As explained here cache entries don't work cross PRs. A cache entry created in one PR won't be accessible from another. However, a cache entry created from master will be accessible to all PRs.

@cburgdorf cburgdorf force-pushed the christoph/ci/mac-win branch 3 times, most recently from 27803e6 to 674c8df Compare October 1, 2020 16:56
@cburgdorf
Copy link
Collaborator Author

@g-r-a-n-t I was fighting with Windows support for some hours but eventually gave up and udpated #62

For now, merging this PR ensures we at least target Linux and Mac.

@cburgdorf cburgdorf requested a review from g-r-a-n-t October 2, 2020 12:49
@cburgdorf cburgdorf changed the title What's up with Mac / Win Run CI against Mac Oct 2, 2020
@cburgdorf cburgdorf merged commit 27dc54f into argotorg:master Oct 2, 2020
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.

3 participants