Remove TODOs around caching workspace discovery (#12510)

## Summary

This happened!
This commit is contained in:
Charlie Marsh 2025-03-27 12:25:09 -04:00 committed by GitHub
parent 882e2f9d3f
commit 7cb85fa08f
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
2 changed files with 2 additions and 6 deletions

View file

@ -48,8 +48,6 @@ impl BuildRequires {
..Default::default()
},
};
// TODO(konsti): Cache workspace discovery.
let Some(project_workspace) =
ProjectWorkspace::from_maybe_project_root(install_path, &discovery, cache).await?
else {

View file

@ -51,8 +51,7 @@ impl RequiresDist {
sources: SourceStrategy,
cache: &WorkspaceCache,
) -> Result<Self, MetadataError> {
// TODO(konsti): Cache workspace discovery.
let discovery_options = DiscoveryOptions {
let discovery = DiscoveryOptions {
stop_discovery_at: git_member.map(|git_member| {
git_member
.fetch_root
@ -66,8 +65,7 @@ impl RequiresDist {
},
};
let Some(project_workspace) =
ProjectWorkspace::from_maybe_project_root(install_path, &discovery_options, cache)
.await?
ProjectWorkspace::from_maybe_project_root(install_path, &discovery, cache).await?
else {
return Ok(Self::from_metadata23(metadata));
};