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

[CI] Add "CI - Integration - Shade on Java *" builds as required #15506

Merged
merged 1 commit into from
May 13, 2022

Conversation

lhotari
Copy link
Member

@lhotari lhotari commented May 9, 2022

Motivation

PIP-156 PR #15264 added new build jobs which should be required checks.
Since required checks are changed, it was first necessary to remove the existing required checks that are were removed in #15264 . This was done in #15496 .
This PR finally adds the new build jobs as required checks.

Modifications

Add CI - Integration - Shade on Java 8, CI - Integration - Shade on Java 11 and CI - Integration - Shade on Java 17 to required checks in .asf.yaml.

Additional details

  • This PR shouldn't be merged before the in-progress PRs are processed, since they might have completed the build using the previous version of the GitHub Actions workflow. Merging this PR will require that the new build version has been used for all PRs.

@lhotari lhotari added doc-not-needed Your PR changes do not impact docs area/ci labels May 9, 2022
@lhotari lhotari self-assigned this May 9, 2022
@lhotari lhotari marked this pull request as ready for review May 12, 2022 17:35
@lhotari lhotari requested a review from michaeljmarshall May 12, 2022 17:35
Copy link
Member

@michaeljmarshall michaeljmarshall left a comment

Choose a reason for hiding this comment

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

LGTM

@merlimat merlimat merged commit 3a1e326 into apache:master May 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/ci doc-not-needed Your PR changes do not impact docs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants