-
Notifications
You must be signed in to change notification settings - Fork 1.7k
feat: Allow cargo check to run on only the current package #16510
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
Conversation
@bors r+ |
☀️ Test successful - checks-actions |
@@ -503,7 +503,7 @@ impl GlobalState { | |||
let mut crate_graph_file_dependencies = FxHashSet::default(); | |||
|
|||
let mut load = |path: &AbsPath| { | |||
let _p = tracing::span!(tracing::Level::INFO, "switch_workspaces::load").entered(); | |||
let _p = tracing::span!(tracing::Level::DEBUG, "switch_workspaces::load").entered(); |
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's up to you, but this change means that hprof
won't pick up on this span.
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.
My base logs became completely useless due to that trace hence the change. Should be fine not to have that included in hprof
Not sure, this only solves the problem for cargo based workspaces. Not for full command overrides. |
cc #13336, #12882