-
Notifications
You must be signed in to change notification settings - Fork 839
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
Improve integration testing docs #1403
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1403 +/- ##
=======================================
Coverage 83.12% 83.13%
=======================================
Files 181 181
Lines 53329 53329
=======================================
+ Hits 44328 44333 +5
+ Misses 9001 8996 -5
Continue to review full report at Codecov.
|
Thanks for the guidance, and we can run the IT test easier. |
@@ -43,7 +43,3 @@ tonic-build = "0.6" | |||
# Pin specific version of the tonic-build dependencies to avoid auto-generated | |||
# (and checked in) arrow.flight.protocol.rs from changing | |||
proc-macro2 = ">1.0.30" | |||
|
|||
#[lib] |
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.
driveby cleanup
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.
LGTM, this will be very useful
Which issue does this PR close?
Re #1398
Rationale for this change
It took me a while to sort out how to run the integration test so I figured I would encode some of that knowledge in docs to assist myself (or others) in the future
What changes are included in this PR?
Update developer docs
Are there any user-facing changes?
No