Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
✨ Source Shopify: add resiliency on some transient errors using the HttpClient #38084
✨ Source Shopify: add resiliency on some transient errors using the HttpClient #38084
Changes from 37 commits
c2b64ee
db2f7ce
b2f44ab
5cf76bb
c9f9718
9e486b8
01c7626
953a756
6e80e7c
bd17a5a
ae30a1f
b6aa6d0
d206b8f
5385b30
202048d
29cef5c
6088f7c
0382003
d652858
2866235
a37dea4
bb56efc
ab522ee
66398ba
805cd29
14861a9
cc45bee
66898dc
9a93b79
1d86e03
ed5ba45
da0cc1b
88a21b9
a2f7186
0025b62
deda781
789306b
7c33dd2
cfcedc0
6df28d1
a72fd04
b604b1d
f6757f4
b705ec8
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
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.
We can also leverage the
stream_slices()
method, to produce 1 slice, or multiple slices, instead of directly injecting thedate-time
str.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.
It feels like we like this isn't reducing the readability of the test and that we should not rely on the source code to create our expectations for the slices that will be produced