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

[7.2.0] Add sep keyword argument to fail #22004

Merged
merged 1 commit into from
Apr 15, 2024

Conversation

bazel-io
Copy link
Member

The argument was already available on 'print' and is necessary for full control over the formatting of error messages that rely on StarlarkValue's debugPrint.

Work towards #20486

Closes #21961.

PiperOrigin-RevId: 625022818
Change-Id: I895b5844d8543f936fc31d367b12bc6291a10bf8

Commit 6b2e0db

The argument was already available on 'print' and is necessary for full control over the formatting of error messages that rely on StarlarkValue's `debugPrint`.

Work towards bazelbuild#20486

Closes bazelbuild#21961.

PiperOrigin-RevId: 625022818
Change-Id: I895b5844d8543f936fc31d367b12bc6291a10bf8
@bazel-io bazel-io added team-Starlark-Interpreter Issues involving the Starlark interpreter used by Bazel awaiting-review PR is awaiting review from an assigned reviewer labels Apr 15, 2024
@bazel-io bazel-io requested a review from a team as a code owner April 15, 2024 18:22
@bazel-io bazel-io requested a review from tetromino April 15, 2024 18:22
@iancha1992 iancha1992 enabled auto-merge April 15, 2024 18:26
@iancha1992 iancha1992 added this pull request to the merge queue Apr 15, 2024
Merged via the queue into bazelbuild:release-7.2.0 with commit 80a3dbe Apr 15, 2024
33 checks passed
@github-actions github-actions bot removed the awaiting-review PR is awaiting review from an assigned reviewer label Apr 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
team-Starlark-Interpreter Issues involving the Starlark interpreter used by Bazel
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants