Merge pull request #33286 from def-/pr-fix-pr-detection-bazel

def- · web-flow · commit f42572d908b5 · 2025-08-12T07:04:57.000+02:00
ci: Switch to LTO on non-PR branches too
diff --git a/ci/mkpipeline.py b/ci/mkpipeline.py
@@ -121,7 +121,7 @@ def main() -> int:
         pipeline.get("env", {}).get("CI_BAZEL_LTO", 0) == 1
         or bool(os.environ["BUILDKITE_TAG"])
         or (
-            os.environ["BUILDKITE_BRANCH"] == "main"
+            not ui.env_is_truthy("BUILDKITE_PULL_REQUEST")
             and args.pipeline in ("nightly", "release-qualification")
         )
         or ui.env_is_truthy("CI_RELEASE_LTO_BUILD")

Original file line number	Diff line number	Diff line change
`@@ -121,7 +121,7 @@ def main() -> int:`
`121`	`121`	`pipeline.get("env", {}).get("CI_BAZEL_LTO", 0) == 1`
`122`	`122`	`or bool(os.environ["BUILDKITE_TAG"])`
`123`	`123`	`or (`
`124`		`- os.environ["BUILDKITE_BRANCH"] == "main"`
	`124`	`+ not ui.env_is_truthy("BUILDKITE_PULL_REQUEST")`
`125`	`125`	`and args.pipeline in ("nightly", "release-qualification")`
`126`	`126`	`)`
`127`	`127`	`or ui.env_is_truthy("CI_RELEASE_LTO_BUILD")`