Skip to content

Fix ArtifactFactory.getPathFromSourceExecPath for external repos#29657

Open
keith wants to merge 1 commit into
bazelbuild:masterfrom
keith:ks/fix-artifactfactory.getpathfromsourceexecpath-for-external-repos
Open

Fix ArtifactFactory.getPathFromSourceExecPath for external repos#29657
keith wants to merge 1 commit into
bazelbuild:masterfrom
keith:ks/fix-artifactfactory.getpathfromsourceexecpath-for-external-repos

Conversation

@keith
Copy link
Copy Markdown
Member

@keith keith commented May 26, 2026

This function is only called for include scanning, which isn't used
widely externally today. Previously if a source file was from an
external repo it would not be correctly found by this function, which
assumed all source files are relative to the workspace.

This function is only called for include scanning, which isn't used
widely externally today. Previously if a source file was from an
external repo it would not be correctly found by this function, which
assumed all source files are relative to the workspace.
@github-actions github-actions Bot added team-Performance Issues for Performance teams awaiting-review PR is awaiting review from an assigned reviewer labels May 26, 2026
@fmeum fmeum requested a review from lberki May 27, 2026 20:17
@github-actions github-actions Bot added the community-reviewed Reviewed by a trusted community contributor label May 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

awaiting-review PR is awaiting review from an assigned reviewer community-reviewed Reviewed by a trusted community contributor team-Performance Issues for Performance teams

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants