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

Removes setting up the JDK in pipelines #27396

Merged
merged 2 commits into from
Feb 4, 2025
Merged

Removes setting up the JDK in pipelines #27396

merged 2 commits into from
Feb 4, 2025

Conversation

jfversluis
Copy link
Member

Description of Change

This is not needed anymore so we can remove this step to not do duplicate work

@jfversluis jfversluis added t/housekeeping ♻︎ area-infrastructure CI, Maestro / Coherency, upstream dependencies/versions labels Jan 28, 2025
@jfversluis jfversluis requested a review from rmarinho January 28, 2025 10:31
@jfversluis jfversluis marked this pull request as ready for review February 4, 2025 21:03
@Copilot Copilot bot review requested due to automatic review settings February 4, 2025 21:03
@jfversluis jfversluis requested a review from a team as a code owner February 4, 2025 21:03
@jfversluis jfversluis requested a review from mattleibow February 4, 2025 21:03

Choose a reason for hiding this comment

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

Copilot reviewed 4 out of 5 changed files in this pull request and generated no comments.

Files not reviewed (1)
  • eng/pipelines/common/setup-jdk.yml: Language not supported
@rmarinho rmarinho merged commit e6f392c into main Feb 4, 2025
124 checks passed
@rmarinho rmarinho deleted the remove-setup-jdk branch February 4, 2025 21:55
rmarinho added a commit that referenced this pull request Feb 10, 2025
This reverts commit e6f392c.
PureWeen pushed a commit that referenced this pull request Feb 10, 2025
This reverts commit e6f392c.
github-actions bot pushed a commit that referenced this pull request Feb 10, 2025
This reverts commit e6f392c.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-infrastructure CI, Maestro / Coherency, upstream dependencies/versions t/housekeeping ♻︎
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants