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.

Sign up for GitHub

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

Jump to bottom

[Bug][ARM] LocationResourceParameter should be azureLocation instead of string #1138

Closed
4 tasks done
live1206 opened this issue Jul 9, 2024 · 0 comments · Fixed by #1139
Closed
4 tasks done

[Bug][ARM] LocationResourceParameter should be azureLocation instead of string #1138

live1206 opened this issue Jul 9, 2024 · 0 comments · Fixed by #1139
Assignees
@live1206
Labels
needs-area

Comments

@live1206
Copy link
Member

live1206 commented Jul 9, 2024

Clear and concise description of the problem

For .NET SDK generation, we would like to have the correct type in spec.
location should be azureLocation instead of string in common-types

Checklist

  • Follow our Code of Conduct
  • Check that this issue is about the Azure libraries for typespec. For feature request in the typespec language or core libraries file it in the TypeSpec repo
  • Read the docs.
  • Check that there isn't already an issue that request the same feature to avoid creating a duplicate.
@live1206 live1206 self-assigned this Jul 9, 2024
@live1206 live1206 mentioned this issue Jul 9, 2024
Merged
@live1206 live1206 changed the title [ARM] LocationResourceParameter should be azureLocation instead of string [Bug][ARM] LocationResourceParameter should be azureLocation instead of string Jul 9, 2024
github-merge-queue bot pushed a commit that referenced this issue Jul 15, 2024
@live1206
location type should be azureLocation instead of string (#1139)
b4df7c5 
resolves #1138
@live1206 live1206 mentioned this issue Jul 17, 2024
Merged
github-merge-queue bot pushed a commit that referenced this issue Jul 19, 2024
@live1206
Use LocationResourceParameter instead of deprecated LocationParameter (…
cfa80d4 
…#1187)

Replace the deprecated `LocationParameter` and apply the fix from
#1138
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@live1206 live1206

Labels
needs-area
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

location type should be azureLocation instead of string live1206/typespec-azure
1 participant
@live1206