Skip to content

Bump sdk version, update tests #771

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

Merged
merged 13 commits into from
Mar 21, 2020

Conversation

MarcoRossignoli
Copy link
Collaborator

@MarcoRossignoli MarcoRossignoli commented Mar 21, 2020

  • Bump sdk to 3.1.*
  • Use @tmds remote executor to avoid to keep in synch with arcade one. Thank's Tom for the tool!

@MarcoRossignoli
Copy link
Collaborator Author

MarcoRossignoli commented Mar 21, 2020

Uh seem not supported on Mac, it's a pity.

@tmds
Copy link

tmds commented Mar 21, 2020

Uh seem not supported on Mac, it's a pity.

Do you want to help make this work? I'll help figure out how to implement it. I don't have a mac myself.

@MarcoRossignoli
Copy link
Collaborator Author

MarcoRossignoli commented Mar 21, 2020

I don't have a mac myself.

Eh the problem is that I don't have Mac too...I work on win and WSL 😞.
We run mac on on CI.

@tmds
Copy link

tmds commented Mar 21, 2020

Damn you, Mac!

I've created an issue for it: tmds/Tmds.ExecFunction#6. Maybe we can find someone willing, and able.

@MarcoRossignoli
Copy link
Collaborator Author

MarcoRossignoli commented Mar 21, 2020

Ok for now I'll skip these test on Mac, historically I never seen any difference for these tests between three plat. Win/Linux should be enough

@MarcoRossignoli MarcoRossignoli changed the title Bump sdk version, update tests [WIP]Bump sdk version, update tests Mar 21, 2020
@MarcoRossignoli MarcoRossignoli changed the title [WIP]Bump sdk version, update tests Bump sdk version, update tests Mar 21, 2020
@MarcoRossignoli MarcoRossignoli merged commit 8ddbb6f into coverlet-coverage:master Mar 21, 2020
@MarcoRossignoli MarcoRossignoli deleted the bumpsdk branch March 21, 2020 16:24
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.

2 participants