-
Notifications
You must be signed in to change notification settings - Fork 8
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
Why the dependence on BenchmarkTools? #105
Comments
Presumably the same goes for the Profile stdlib |
Good catch, must be an oversight on my part. Will deal with this at some point 👍 |
Where abouts in the code did you find the |
Adding Tapir to my test environment back when it wasn't registered happened after instantiation, and so I saw all the packages that were added to the manifest. The ones that struck me as out of place were Profile and BenchmarkTools, but maybe Profile was an indirect dependency (eg of BenchmarkTools), I didn't double-check. |
Cool -- yeah, I think you're correct re Profile being an indirect dep of Tapir. Thanks for confirming. |
I looked for
@b
in the package and it only shows up in benchmarking or testing. Any reason to keep BenchmarkTools in the package's dependencies?The text was updated successfully, but these errors were encountered: