-
Notifications
You must be signed in to change notification settings - Fork 1.3k
[AutoPR @azure-arm-impactreporting]-generated-from-SDK Generation - JS-5274250 #35801
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
base: main
Are you sure you want to change the base?
[AutoPR @azure-arm-impactreporting]-generated-from-SDK Generation - JS-5274250 #35801
Conversation
…l', API Version: 2024-05-01-preview, SDK Release Type: beta, and CommitSHA: 'f9b00d9100abf7a317573f66dcf1fac5ea467b3d' in SpecRepo: 'https://github.com/Azure/azure-rest-api-specs' Pipeline run: https://dev.azure.com/azure-sdk/internal/_build/results?buildId=5274250 Refer to https://eng.ms/docs/products/azure-developer-experience/develop/sdk-release/sdk-release-prerequisites to prepare for SDK release.
There was a problem hiding this 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 is an auto-generated SDK update for the Azure Impact Reporting SDK, updating from an earlier commit to commit f9b00d9100abf7a317573f66dcf1fac5ea467b3d
in the Azure REST API specs. The update restructures the codebase to better separate API operations into their own modules and adds new functionality.
- Restructures API operations into separate modules with dedicated
operations.ts
andoptions.ts
files - Adds URL template expansion functionality and cloud setting helpers
- Updates models with new types and reorders export declarations
- Introduces new sample files demonstrating SDK usage
Reviewed Changes
Copilot reviewed 53 out of 55 changed files in this pull request and generated 2 comments.
Show a summary per file
File | Description |
---|---|
ci.mgmt.yml |
Adds CI pipeline path configuration |
tsp-location.yaml |
Updates commit reference |
static-helpers/ |
Adds new URL template and cloud setting helper utilities |
restorePollerHelpers.ts |
Updates import paths and expected status codes |
models/ |
Major restructuring of model definitions and exports |
api/ |
Refactors API structure into separate operation modules |
classic/ |
Updates classic API wrappers to use new operation imports |
samples-dev/ |
Adds comprehensive sample code files |
sdk/impactreporting/arm-impactreporting/src/static-helpers/urlTemplate.ts
Show resolved
Hide resolved
sdk/impactreporting/arm-impactreporting/src/static-helpers/cloudSettingHelpers.ts
Show resolved
Hide resolved
): ImpactContext { | ||
const endpointUrl = options.endpoint ?? options.baseUrl ?? "https://management.azure.com"; | ||
const endpointUrl = | ||
options.endpoint ?? getArmEndpoint(options.cloudSetting) ?? "https://management.azure.com"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@wanlwanl Could you prioritize the issue? It impacts all Modular service release - Azure/azure-sdk-tools#11825
@v-jiaodi the version in below is wrong, please update it manually. you could refer the version in changelog.
const userAgentInfo = `azsdk-js-arm-impactreporting/1.0.0-beta.1`;
API Change CheckAPIView identified API level changes in this PR and created the following API reviews |
Configurations: 'specification/impact/Impact.Management/tspconfig.yaml', API Version: 2024-05-01-preview, SDK Release Type: beta, and CommitSHA: 'f9b00d9100abf7a317573f66dcf1fac5ea467b3d' in SpecRepo: 'https://github.com/Azure/azure-rest-api-specs' Pipeline run: https://dev.azure.com/azure-sdk/internal/_build/results?buildId=5274250 Refer to https://eng.ms/docs/products/azure-developer-experience/develop/sdk-release/sdk-release-prerequisites to prepare for SDK release.
Release Plan Details