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

Switch from travis to GitHub Actions for CI testing #2552

Merged
merged 2 commits into from
Nov 20, 2020
Merged

Switch from travis to GitHub Actions for CI testing #2552

merged 2 commits into from
Nov 20, 2020

Conversation

quinnj
Copy link
Member

@quinnj quinnj commented Nov 20, 2020

No description provided.

@bkamins
Copy link
Member

bkamins commented Nov 20, 2020

For me the set of test coverage looks good. We drop OSx but I think it is acceptable.

@bkamins
Copy link
Member

bkamins commented Nov 20, 2020

Let us wait for @nalimilan to comment before making a final decision.

@bkamins bkamins added the CI label Nov 20, 2020
@bkamins bkamins added this to the 1.0 milestone Nov 20, 2020
Copy link
Member

@nalimilan nalimilan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If that works it's fine with me!

@bkamins
Copy link
Member

bkamins commented Nov 20, 2020

OK - let us wait for @quinnj to go back to this and merge if he is happy 😄.

@quinnj quinnj merged commit dd87a61 into master Nov 20, 2020
@quinnj quinnj deleted the jq/ga branch November 20, 2020 16: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.

3 participants