Closed alanaasmaa closed 3 years ago
@alanaasmaa, do you want to continue with this PR? There aren't changes after your force push...
@alanaasmaa, do you want to continue with this PR? There aren't changes after your force push...
Yes i would, I'm not very good with rebasing :D EDIT: Now there are those changes again, maybe some day i will learn how to use rebase correctly.
Merging #74 (1de873f) into master (a717a61) will increase coverage by
0.02%
. The diff coverage is100.00%
.
@@ Coverage Diff @@
## master #74 +/- ##
============================================
+ Coverage 82.06% 82.08% +0.02%
Complexity 370 370
============================================
Files 22 22
Lines 853 854 +1
============================================
+ Hits 700 701 +1
Misses 153 153
Impacted Files | Coverage Δ | Complexity Δ | |
---|---|---|---|
src/InfluxDB2/Client.php | 93.54% <ø> (ø) |
12.00 <0.00> (ø) |
|
src/InfluxDB2/DefaultApi.php | 84.37% <100.00%> (+0.24%) |
17.00 <10.00> (ø) |
Continue to review full report at Codecov.
Legend - Click here to learn more
Δ = absolute <relative> (impact)
,ø = not affected
,? = missing data
Powered by Codecov. Last update a717a61...1de873f. Read the comment docs.
Closes #72
Proposed Changes
I made it possible to change guzzleHttp timeout. Sadly i couldn't get tests to work so I can't run them. But I did manual testing, i'm only using Client and
createWriteApi
.Checklist
make test
completes successfully