Merge pull request #1 from microsoft/users/tedchamb/dev

Update client methods to take a single struct arg to allow for additional args in future with no compatibility breaks.

.gitattributes

@@ -1,6 +1,7 @@
 # Set the default behavior (used when a rule below doesn't match)
 * text=auto
 
+# Go files
 *.go text eol=lf
 *.mod text eol=lf
 *.sum text eol=lf

README.md

@@ -28,13 +28,13 @@ func main() {
     if err != nil {
         log.Fatal(err)
     }
-    
+
     // Get a list of all team projects
-    teamProjectReferences, err := coreClient.GetProjects(ctx, nil, nil, nil, nil, nil)
+    teamProjectReferences, err := coreClient.GetProjects(ctx, core.GetProjectsArgs{})
     if err != nil {
         log.Fatal(err)
     }
-    
+
     // Print the list of team project names for your organization
     for index, teamProjectReference := range *teamProjectReferences {
         log.Printf("Name[%v] = %v", index, *teamProjectReference.Name)

azureDevops/accounts/client.go

@@ -33,20 +33,16 @@ func NewClient(ctx context.Context, connection azureDevops.Connection) (*Client,
 }
 
 // Get a list of accounts for a specific owner or a specific member.
-// ctx
-// ownerId (optional): ID for the owner of the accounts.
-// memberId (optional): ID for a member of the accounts.
-// properties (optional)
-func (client Client) GetAccounts(ctx context.Context, ownerId *uuid.UUID, memberId *uuid.UUID, properties *string) (*[]Account, error) {
+func (client Client) GetAccounts(ctx context.Context, args GetAccountsArgs) (*[]Account, error) {
     queryParams := url.Values{}
-    if ownerId != nil {
-        queryParams.Add("ownerId", (*ownerId).String())
+    if args.OwnerId != nil {
+        queryParams.Add("ownerId", (*args.OwnerId).String())
     }
-    if memberId != nil {
-        queryParams.Add("memberId", (*memberId).String())
+    if args.MemberId != nil {
+        queryParams.Add("memberId", (*args.MemberId).String())
     }
-    if properties != nil {
-        queryParams.Add("properties", *properties)
+    if args.Properties != nil {
+        queryParams.Add("properties", *args.Properties)
     }
     locationId, _ := uuid.Parse("229a6a53-b428-4ffb-a835-e8f36b5b4b1e")
     resp, err := client.Client.Send(ctx, http.MethodGet, locationId, "5.1", nil, queryParams, nil, "", "application/json", nil)
@@ -59,3 +55,13 @@ func (client Client) GetAccounts(ctx context.Context, ownerId *uuid.UUID, member
     return &responseValue, err
 }
 
+// Arguments for the GetAccounts function
+type GetAccountsArgs struct {
+    // (optional) ID for the owner of the accounts.
+    OwnerId *uuid.UUID
+    // (optional) ID for a member of the accounts.
+    MemberId *uuid.UUID
+    // (optional)
+    Properties *string
+}
+