Skip to content
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

[8.1.0] Use shorthand display form in Label#debugPrint #24855

Open
wants to merge 1 commit into
base: release-8.1.0
Choose a base branch
from

Conversation

bazel-io
Copy link
Member

@bazel-io bazel-io commented Jan 8, 2025

This makes the output, which is meant to be friendly to humans, more concise. If users want //foo to be printed as //foo:foo, they can specify sep = "" and also print :foo, but stripping the name off is not possible without this change.

Closes #24840.

PiperOrigin-RevId: 713207601
Change-Id: I474fc4eccb1419304368a5c90a22c13b2197b699

Commit e0fada9

This makes the output, which is meant to be friendly to humans, more concise. If users want `//foo` to be `print`ed as `//foo:foo`, they can specify `sep = ""` and also print `:foo`, but stripping the name off is not possible without this change.

Closes bazelbuild#24840.

PiperOrigin-RevId: 713207601
Change-Id: I474fc4eccb1419304368a5c90a22c13b2197b699
@bazel-io bazel-io requested a review from a team as a code owner January 8, 2025 10:02
@bazel-io bazel-io added the awaiting-review PR is awaiting review from an assigned reviewer label Jan 8, 2025
@bazel-io bazel-io requested a review from meteorcloudy January 8, 2025 10:02
@meteorcloudy meteorcloudy enabled auto-merge (squash) January 8, 2025 13:16
@meteorcloudy meteorcloudy disabled auto-merge January 8, 2025 13:16
@iancha1992 iancha1992 added the team-Configurability platforms, toolchains, cquery, select(), config transitions label Jan 8, 2025
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 team-Configurability platforms, toolchains, cquery, select(), config transitions
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants