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
TypedSDK
- update all int
struct properties to int64
#25910
base: main
Are you sure you want to change the base?
Conversation
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.
Apart from a couple of overlooked int32s, LGTM 🐐
@@ -435,7 +436,7 @@ func (r ThreatIntelligenceIndicator) Create() sdk.ResourceFunc { | |||
props.Pattern = &patternValue | |||
|
|||
if model.Confidence != -1 { | |||
props.Confidence = utils.Int32(model.Confidence) | |||
props.Confidence = pointer.To(int32(model.Confidence)) |
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.
props.Confidence = pointer.To(int32(model.Confidence)) | |
props.Confidence = pointer.To(int64(model.Confidence)) |
@@ -561,7 +562,7 @@ func (r ThreatIntelligenceIndicator) Update() sdk.ResourceFunc { | |||
if model.Confidence == -1 { | |||
properties.Confidence = nil | |||
} else { | |||
properties.Confidence = &model.Confidence | |||
properties.Confidence = pointer.To(int32(model.Confidence)) |
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.
properties.Confidence = pointer.To(int32(model.Confidence)) | |
properties.Confidence = pointer.To(int64(model.Confidence)) |
Community Note
Description
This change updates all
int
references in the Typed resources to beint64
to align with the values in the SDKs provided bygo-azure-sdk
PR Checklist
For example: “
resource_name_here
- description of change e.g. adding propertynew_property_name_here
”Changes to existing Resource / Data Source
Change Log
Below please provide what should go into the changelog (if anything) conforming to the Changelog Format documented here.
TypedSDK
- update allint
struct properties toint64