Skip to content

Conversation

electrum
Copy link
Member

@electrum electrum commented Oct 1, 2025

Release notes

(x) This is not user-visible or is docs only, and no release notes are required.

Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR removes JDK11-specific classifier configurations for the oauth2-oidc-sdk dependency across multiple modules. The changes clean up dependency management by removing redundant exclusions and classifier specifications that are no longer needed.

  • Removes the jdk11 classifier from oauth2-oidc-sdk dependencies
  • Eliminates unnecessary exclusions of oauth2-oidc-sdk from Azure dependencies
  • Simplifies dependency management by removing redundant runtime dependencies

Reviewed Changes

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

File Description
pom.xml Removes jdk11 classifier from main oauth2-oidc-sdk dependency definition
plugin/trino-exchange-filesystem/pom.xml Removes oauth2-oidc-sdk exclusions and runtime dependency with jdk11 classifier
lib/trino-filesystem-azure/pom.xml Removes oauth2-oidc-sdk exclusions and runtime dependency with jdk11 classifier
core/trino-main/pom.xml Removes jdk11 classifier from oauth2-oidc-sdk dependency

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@wendigo
Copy link
Contributor

wendigo commented Oct 1, 2025

@sourcery-ai review

@SourceryAI
Copy link

Hi @wendigo! 👋

Only authors and team members can run @sourcery-ai commands on public repos.

If you are a team member, install the @sourcery-ai bot to get access ✨

@wendigo wendigo merged commit 1c81bb1 into master Oct 1, 2025
206 of 209 checks passed
@wendigo wendigo deleted the electrum/azure branch October 1, 2025 08:36
@github-actions github-actions bot added this to the 478 milestone Oct 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

3 participants