-
Notifications
You must be signed in to change notification settings - Fork 263
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
Add ArticlesService in the client #309
Conversation
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.
Looks good, thanks for the PR!
one minor mini nit, can you properly group the imports in the test file plz?
articles_test.go
Outdated
import ( | ||
"context" | ||
"fmt" | ||
"github.com/jarcoal/httpmock" |
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.
nit: break out the 3rd party import in a separate section
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.
thanks! done.
Implement ArticlesService for managing Shopify articles, including CRUD operations, tag related operations, and article count.
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #309 +/- ##
==========================================
- Coverage 99.87% 99.11% -0.76%
==========================================
Files 52 53 +1
Lines 2320 2363 +43
==========================================
+ Hits 2317 2342 +25
- Misses 2 20 +18
Partials 1 1
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Can someone merge this? I was just about to create a new issue about the missing Articles services |
Implement ArticlesService for managing Shopify articles, including CRUD operations, tag related operations, and article count.