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

Update Travis References #52

Merged
merged 8 commits into from Jun 8, 2021
Merged

Update Travis References #52

merged 8 commits into from Jun 8, 2021

Conversation

rpcrimi
Copy link
Collaborator

@rpcrimi rpcrimi commented Jun 7, 2021

Pull Request Template

Updating references of travis-ci.org to travis-ci.com

Description

On June 30, travis-ci.org will shut off. The new url will be travis-ci.com. I've done the migration to the new url already. This pr updates any references to travis-ci.org to travis-ci.com

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added reviewers to the PR.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

Copy link
Contributor

@cfrazier91 cfrazier91 left a comment

Choose a reason for hiding this comment

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

LGTM (assuming you have tested) @rpcrimi !

@rpcrimi
Copy link
Collaborator Author

rpcrimi commented Jun 7, 2021

@cfrazier91 @denver1117 it looks like the spark 2.4 build has been failing for a while. i attempted to fix it here but couldn't make any progress.

Copy link
Contributor

@denver1117 denver1117 left a comment

Choose a reason for hiding this comment

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

Looks good. Tests should pass when this PR is merged. We have to occasionally bump the spark version for the test suite since the minor version number changes from time to time and the remove the legacy minor version from the distribution repository. I just go here to find it.

@rpcrimi
Copy link
Collaborator Author

rpcrimi commented Jun 8, 2021

Looks good. Tests should pass when this PR is merged. We have to occasionally bump the spark version for the test suite since the minor version number changes from time to time and the remove the legacy minor version from the distribution repository. I just go here to find it.

Sounds good. I will approve that PR after merging this one.

@rpcrimi rpcrimi merged commit 14a3006 into master Jun 8, 2021
@rpcrimi rpcrimi deleted the update_travis_references branch June 8, 2021 15:25
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