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

Changelog Tool Bumps Stable Version for Preview API with Latest Modular CodeGen #8522

Closed
Tracked by #2638 ...
kazrael2119 opened this issue Jul 1, 2024 · 2 comments
Closed
Tracked by #2638 ...
Assignees
Labels
P0 SDK Automation Issues related to sdk automation

Comments

@kazrael2119
Copy link
Contributor

this pr is generated with modular
Azure/azure-sdk-for-js#30102
and the generated version is 1.0.0-beta.1
but after usiing changelog tool, it becomes 1.0.0

@wanlwanl
Copy link
Member

wanlwanl commented Jul 2, 2024

The bug is introduced by new codegen pr. The way to define default api version is changed.
extracting the api version should also consider version of codegen.

e.g. new api-version at client level is defined here
while old one is here

tasks:

  • update to follow the new pattern on new codegen version while follow old pattern on old codegen pattern
  • add changelog generation check to codegen repo

@wanlwanl wanlwanl added SDK Automation Issues related to sdk automation P0 labels Jul 9, 2024
@wanlwanl wanlwanl changed the title [JS] generate the wrong version if it is a FirstBeta package generated with modular [JS] Changelog Tool Bumps Package Version to 1.0.0 in with Preview API Version in Latest autorest.typescript Modular CodeGen Jul 9, 2024
@wanlwanl wanlwanl changed the title [JS] Changelog Tool Bumps Package Version to 1.0.0 in with Preview API Version in Latest autorest.typescript Modular CodeGen [JS] Changelog Tool Bumps Stable Version for Preview API Version with Latest Modular CodeGen Jul 9, 2024
@wanlwanl wanlwanl changed the title [JS] Changelog Tool Bumps Stable Version for Preview API Version with Latest Modular CodeGen [JS] Changelog Tool Bumps Stable Version for Preview API with Latest Modular CodeGen Jul 9, 2024
@wanlwanl wanlwanl changed the title [JS] Changelog Tool Bumps Stable Version for Preview API with Latest Modular CodeGen Changelog Tool Bumps Stable Version for Preview API with Latest Modular CodeGen Jul 9, 2024
@wanlwanl
Copy link
Member

PR to fix: #8671

wanlwanl added a commit that referenced this issue Jul 22, 2024
# Issue
#8522 

# Problem
The bug is introduced by new codegen [pr](Azure/autorest.typescript@b0853b2). The way to define default api version is changed.
extracting the api version should also consider version of codegen.

e.g. new api-version at client level is defined [here](https://github.com/Azure/azure-sdk-for-js/blob/main/sdk/mongocluster/arm-mongocluster/src/rest/documentDBClient.ts#L23)
while old one is:
- [here](https://github.com/Azure/azure-sdk-for-js/blob/06716722818f5838cc10a9e5644b7ba9f32089d5/sdk/face/ai-vision-face-rest/src/faceClient.ts#L26)
- [another](https://github.com/Azure/azure-sdk-for-js/blob/06716722818f5838cc10a9e5644b7ba9f32089d5/sdk/openai/openai-rest/src/openAIClient.ts#L22)

# Solution
1. try to detect in new client, return api-version if found
2. fallback to detect old client, return whatever found
WhitShake pushed a commit to WhitShake/azure-sdk-tools that referenced this issue Jul 24, 2024
# Issue
Azure#8522 

# Problem
The bug is introduced by new codegen [pr](Azure/autorest.typescript@b0853b2). The way to define default api version is changed.
extracting the api version should also consider version of codegen.

e.g. new api-version at client level is defined [here](https://github.com/Azure/azure-sdk-for-js/blob/main/sdk/mongocluster/arm-mongocluster/src/rest/documentDBClient.ts#L23)
while old one is:
- [here](https://github.com/Azure/azure-sdk-for-js/blob/06716722818f5838cc10a9e5644b7ba9f32089d5/sdk/face/ai-vision-face-rest/src/faceClient.ts#L26)
- [another](https://github.com/Azure/azure-sdk-for-js/blob/06716722818f5838cc10a9e5644b7ba9f32089d5/sdk/openai/openai-rest/src/openAIClient.ts#L22)

# Solution
1. try to detect in new client, return api-version if found
2. fallback to detect old client, return whatever found
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
P0 SDK Automation Issues related to sdk automation
Projects
None yet
Development

No branches or pull requests

2 participants