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

Run CI against Mac #63

Merged
merged 1 commit into from
Oct 2, 2020
Merged

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

@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 changed the title What's up with Mac / Win Run CI against Mac Oct 2, 2020
@cburgdorf cburgdorf merged commit 27dc54f into ethereum: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.

None yet

3 participants