Skip to content
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

chore: SimpleClient -> Standard Client #210

Conversation

ethenotethan
Copy link
Collaborator

Fixes Issue

Fixes #

Changes proposed

Screenshots (Optional)

Note to reviewers

@ethenotethan ethenotethan requested a review from samlaf December 5, 2024 22:17
@samlaf samlaf requested a review from bxue-l2 December 6, 2024 16:11
@ethenotethan ethenotethan changed the title chore: SimpleClient -> General Client chore: SimpleClient -> Standard Client Dec 7, 2024
Copy link
Collaborator

@samlaf samlaf left a comment

Choose a reason for hiding this comment

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

LGTM! small nit is that I wouldnt use "std" anywhere because that sounds like its the standard library. But not a biggy

@ethenotethan ethenotethan merged commit 553417e into feat--separate-module-for-simple-client Dec 7, 2024
1 check passed
@ethenotethan ethenotethan deleted the epociask--chore-rename-simple-client branch December 7, 2024 14:54
ethenotethan added a commit that referenced this pull request Dec 9, 2024
* feat: separate simple client into its own module

This will make it easier to import it into other projects without importing all of the other dependencies of our root go.mod.
We will first merge this PR with a replace directive in the main go.mod, then release v0.1.0 of the new client module, and then
make a new PR to use that version in the main go.mod file.

* chore: SimpleClient -> Standard Client (#210)

---------

Co-authored-by: Ethen <[email protected]>
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