-
Notifications
You must be signed in to change notification settings - Fork 73
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
Use cli messaging for cache #1127
Conversation
This is how benchmark results would change (along with a 95% confidence interval in relative change) if e8836fd is merged into main:
Further explanation regarding interpretation and methodology can be found in the documentation. |
Looks good, thanks. If you want, you could add links to a few more places:
Also, if you search for https://github.com/, you find a few more places where we can use the |
* Fix with autolinking in vignette * Update a test that I broke.
I fixed a test. Line 36 in e8836fd
but it wouldn't really fit in the context. |
Codecov Report
@@ Coverage Diff @@
## main #1127 +/- ##
=======================================
Coverage 92.18% 92.18%
=======================================
Files 46 46
Lines 2650 2650
=======================================
Hits 2443 2443
Misses 207 207
|
This is how benchmark results would change (along with a 95% confidence interval in relative change) if e8836fd is merged into main:
Further explanation regarding interpretation and methodology can be found in the documentation. |
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
Add a link for RStudio users