Skip to content

chore: Refactor client call to a single place - #71

Merged
dbolson merged 3 commits into
mainfrom
refactor-client
Mar 26, 2024
Merged

chore: Refactor client call to a single place#71
dbolson merged 3 commits into
mainfrom
refactor-client

Conversation

@dbolson

@dbolson dbolson commented Mar 26, 2024

Copy link
Copy Markdown
Contributor

No description provided.

Comment thread internal/client/client.go
@@ -0,0 +1,14 @@
package client

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I made this singular to hint that there's only one client, but I could change it to clients for consistency if preferred.

@k3llymariee k3llymariee left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I also have one in the members package now - I can clean this up in a follow up PR if you don't wanna do that here.

@dbolson

dbolson commented Mar 26, 2024

Copy link
Copy Markdown
Contributor Author

@k3llymariee I made the changes in this PR.

@dbolson
dbolson merged commit a09be6f into main Mar 26, 2024
@dbolson
dbolson deleted the refactor-client branch March 26, 2024 20:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants