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

Author: tedchamb

.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

@@ -18,27 +18,27 @@ func main() {
     organizationUrl := "https://dev.azure.com/myorg"  // todo: replace value with your organization url
     personalAccessToken := "XXXXXXXXXXXXXXXXXXXXXXX"  // todo: replace value with your PAT
 
-    // Create a connection to your organization
-    connection := azureDevops.NewConnection(organizationUrl, personalAccessToken)
-
-    ctx := context.Background()
-
-    // Create a client to interact with the Core area
-    coreClient, err := core.NewClient(ctx, *connection)
-    if err != nil {
-        log.Fatal(err)
-    }
-    
-    // Get a list of all team projects
-    teamProjectReferences, err := coreClient.GetProjects(ctx, nil, nil, nil, nil, nil)
-    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)
-    }
+	// Create a connection to your organization
+	connection := azureDevops.NewConnection(organizationUrl, personalAccessToken)
+
+	ctx := context.Background()
+
+	// Create a client to interact with the Core area
+	coreClient, err := core.NewClient(ctx, *connection)
+	if err != nil {
+		log.Fatal(err)
+	}
+
+	// Get a list of all team projects
+	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
+}
+