Skip to content

Commit 5b8dcb0

Browse files
committed
Make it clearer why long CI times are worth it
1 parent 3a999d2 commit 5b8dcb0

File tree

1 file changed

+6
-4
lines changed

1 file changed

+6
-4
lines changed

src/infra/docs/rustc-ci.md

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -90,11 +90,13 @@ hours for a full run)[^1], means we can’t merge too many PRs in a single day,
9090
single failure greatly impacts our throughput for the day. The maximum number
9191
of PRs we can merge in a day is around 8.
9292

93-
The large CI run times and requirement for a large builder pool is worth it because it:
93+
The large CI run times and requirement for a large builder pool is largely due to the
94+
fact that full release artifacts are built in the `dist-` builders. This is worth it
95+
because these release artifacts:
9496

95-
- allows perf testing even at a later date
96-
- allows bisection when bugs are discover later
97-
- ensures release quality since if we're always releasing, we catch problems very early
97+
- allow perf testing even at a later date
98+
- allow bisection when bugs are discovered later
99+
- ensure release quality since if we're always releasing, we can catch problems early
98100

99101
Bors [runs on ecs](https://github.com/rust-lang/simpleinfra/blob/master/terraform/bors/app.tf) and uses a sqlite database running in a volume as storage.
100102

0 commit comments

Comments
 (0)