-
Notifications
You must be signed in to change notification settings - Fork 12
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
feat: add support for enterprise and child organizations #77
base: main
Are you sure you want to change the base?
Conversation
92cc2a3
to
1db1e5e
Compare
1db1e5e
to
98e0cbb
Compare
logdna/common_test.go
Outdated
copy(opts, args) | ||
sk, ul := opts[0], opts[1] | ||
sk, ul, tp := opts[0], opts[1], opts[2] |
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.
What do you think of renaming these variables to something that imparts meaning? From the following fmt.Sprintf
, it looks like sk
=> serviceKey
would make sense. I have no idea what the other two variables would be (the crux of the issue).
func buildSchemaMap(a []TerraformInfo) map[string]*schema.Resource { | ||
m := make(map[string]*schema.Resource) | ||
|
||
for _, e := range a { | ||
m[e.name] = e.schema | ||
} | ||
|
||
return m | ||
} |
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.
func buildSchemaMap(a []TerraformInfo) map[string]*schema.Resource { | |
m := make(map[string]*schema.Resource) | |
for _, e := range a { | |
m[e.name] = e.schema | |
} | |
return m | |
} | |
func buildSchemaMap(tfInfoList []TerraformInfo) map[string]*schema.Resource { | |
schemaMap := make(map[string]*schema.Resource) | |
for _, info := range tfInfoList { | |
schemaMap[info.name] = info.schema | |
} | |
return schemaMap | |
} |
Mainly, the a
doesn't impart any information to the reader. Renaming the m
and e
variables would also really help the next person to look at this code.
1b28fa9
to
3a399a0
Compare
3a399a0
to
dd9048b
Compare
@jakedipity Mezmo laid me off on January 19. Please request a review from someone else. |
@jakedipity , you could add @seeruyy |
dd9048b
to
2852809
Compare
Yeah sorry, that was a misclick on my part. |
This begins support for terraform management of enterprise and child organizations: * Allows the provider to accept an enterprise key to indicate an enterprise organization. * Enterprise organizations can attach and detach child organizations. Ref: LOG-13116 Signed-off-by: Jacob Hull <[email protected]>
d85de8c
to
8441d12
Compare
This begins support for terraform management of enterprise and child organizations:
Ref: LOG-13116
Signed-off-by: Jacob Hull [email protected]