-
Notifications
You must be signed in to change notification settings - Fork 2k
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
Clean up POM dependencies #42469
Open
alzimmermsft
wants to merge
2
commits into
Azure:main
Choose a base branch
from
alzimmermsft:AzPom_CleanUpDependenciesFromAzureCoreTest
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Clean up POM dependencies #42469
alzimmermsft
wants to merge
2
commits into
Azure:main
from
alzimmermsft:AzPom_CleanUpDependenciesFromAzureCoreTest
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
alzimmermsft
requested review from
weidongxu-microsoft,
haolingdong-msft,
XiaofeiCao,
ArthurMa1978,
jeanbisutti,
trask,
ramthi,
heyams,
harsimar,
MikeyMCZ,
jairmyree,
vcolin7,
ibrahimrabab,
seanmcc-msft,
ibrandes,
kyleknap,
vicancy,
chenkennt,
zackliu,
anuchandy,
conniey,
lmolkova,
a team,
srnagar,
MichaelZp0,
ChristopherManthei,
vxfield and
yifan-zhou922
as code owners
October 18, 2024 20:51
github-actions
bot
added
Event Grid
Event Hubs
Health Deidentification
Image Analysis
KeyVault
Load Testing
Mgmt
This issue is related to a management-plane library.
Mixed Reality Authentication
Models Repository
Monitor
Monitor, Monitor Ingestion, Monitor Query
OpenAI
OpenTelemetry
OpenTelemetry instrumentation
Purview
Remote Rendering
Schema Registry
Search
Service Bus
Storage
Storage Service (Queues, Blobs, Files)
Tables
Text Translation
WebPubSub
labels
Oct 18, 2024
mssfang
approved these changes
Oct 18, 2024
anuchandy
approved these changes
Oct 18, 2024
API change check API changes are not detected in this pull request. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
AI Model Inference
App Configuration
Azure.ApplicationModel.Configuration
Attestation
Azure.Core.AMQP
azure-core-amqp
Azure.Core
azure-core
Azure.Identity
Azure Quantum
azure-spring
All azure-spring related issues
Batch
Cognitive - Anomaly Detector
Cognitive - Face
Cognitive - Form Recognizer
Cognitive - Health Insights
Cognitive - Metrics Advisor
Cognitive - Personalizer
Cognitive - Text Analytics
Communication - Call Automation
Communication - Calling Server
Communication - Chat
Communication - Email
Communication - Identity
Communication - Phone Numbers
Communication - Rooms
Communication - SMS
DevCenter
Digital Twins
Document Intelligence
Event Grid
Event Hubs
Health Deidentification
Image Analysis
KeyVault
Load Testing
Mgmt
This issue is related to a management-plane library.
Mixed Reality Authentication
Models Repository
Monitor
Monitor, Monitor Ingestion, Monitor Query
OpenAI
OpenTelemetry
OpenTelemetry instrumentation
Purview
Remote Rendering
Schema Registry
Search
Service Bus
Storage
Storage Service (Queues, Blobs, Files)
Tables
Text Translation
WebPubSub
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Removes the
azure-core-http-jdk-httpclient
exclusion in theazure-core-test
dependency as the issue which caused Java 8 runs to fail when attempting to load the JDK HttpClient, which is compiled targeting Java 11 bytecode, was fixed (refer). This also allowed for the removal of the JDK 12+ profile which added the JDK HttpClient implementation as a dependency.While inspecting all
azure-core-test
usages in POMs, POMs were verified for their design and other build, profile, and dependencies were removed.codesnippet-maven-plugin
were removed when they matched the default configuration.All SDK Contribution checklist:
General Guidelines and Best Practices
Testing Guidelines