Skip to content

Add OSC 8 escape sequence for BES URL - #31018

Open
keith wants to merge 1 commit into
bazelbuild:masterfrom
keith:ks/add-osc-8-escape-sequence-for-bes-url
Open

Add OSC 8 escape sequence for BES URL#31018
keith wants to merge 1 commit into
bazelbuild:masterfrom
keith:ks/add-osc-8-escape-sequence-for-bes-url

Conversation

@keith

@keith keith commented Sep 5, 2026

Copy link
Copy Markdown
Member

Terminals try to make URLs clickable but that can break when the URLs
wrap because of terminal width. This wraps the BES URL with the OSC 8
escape sequence which terminals will prefer over their default logic to
find URLs.

@github-actions github-actions Bot added team-Core Skyframe, bazel query, BEP, options parsing, bazelrc awaiting-review PR is awaiting review from an assigned reviewer labels Sep 5, 2026
@github-actions github-actions Bot added the community-reviewed Reviewed by a trusted community contributor label Sep 5, 2026
@keith
keith force-pushed the ks/add-osc-8-escape-sequence-for-bes-url branch from ba84bbc to 633587d Compare September 10, 2026 16:33
@keith

keith commented Sep 10, 2026

Copy link
Copy Markdown
Member Author

@meisterT I just rebased this on 920afba, want to take a look?

Terminals try to make URLs clickable but that can break when the URLs
wrap because of terminal width. This wraps the BES URL with the OSC 8
escape sequence which terminals will prefer over their default logic to
find URLs.
@keith
keith force-pushed the ks/add-osc-8-escape-sequence-for-bes-url branch from 633587d to 41946f2 Compare September 10, 2026 16:35
@meisterT meisterT added copybara:import Trigger copybara to import the PR and removed awaiting-review PR is awaiting review from an assigned reviewer labels Sep 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

community-reviewed Reviewed by a trusted community contributor copybara:import Trigger copybara to import the PR team-Core Skyframe, bazel query, BEP, options parsing, bazelrc

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants