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

DRILL-7543: Use GitHub Actions for CI #1966

Merged
merged 1 commit into from Feb 5, 2020

Conversation

vvysotskyi
Copy link
Member

@vvysotskyi vvysotskyi commented Feb 4, 2020

DRILL-7543: Use GitHub Actions for CI

Description

Removes usage of previously used Travis and CircleCI and added GitHub Actions to run full test suites for several JDK versions on the push to PR or branch.

Documentation

NA

Testing

Ran job on private brach, all tests pass.

@vvysotskyi vvysotskyi force-pushed the DRILL-7543 branch 2 times, most recently from dd3b24d to c2e273e Compare February 4, 2020 14:24
Copy link
Member

@arina-ielchiieva arina-ielchiieva left a comment

Choose a reason for hiding this comment

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

Really nice, one minor comment.

# limitations under the License.
#

# This workflow is triggered for every push events to the master branch or pull request.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
# This workflow is triggered for every push events to the master branch or pull request.
# This workflow is triggered for each push event to the master branch or pull request.

Copy link
Member Author

Choose a reason for hiding this comment

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

Thanks, fixed.

@arina-ielchiieva
Copy link
Member

+1, thanks for the changes, they will be really useful for the Drill project.

@arina-ielchiieva arina-ielchiieva merged commit 1288e71 into apache:master Feb 5, 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

2 participants