New issue
Advanced search Search tips
Note: Color blocks (like █ or █) mean that a user may not be available. Tooltip shows the reason.

Issue 906144 link

Starred by 1 user
Status: Available
Owner: ----
Cc:
█ iannucci@chromium.org
iannu...@google.com
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

Make "recipes.py test run" return non-zero if README.recipes.md is out of date.

Project Member Reported by bpastene@google.com, Nov 16 
As is, changes that affect README.recipes.md but don't regen it can land. This causes other unrelated CLs from picking up those changes to the doc.

ie: see https://chromium-review.googlesource.com/c/chromium/tools/build/+/1337599 which didn't regen it. Then see the later https://chromium-review.googlesource.com/c/chromium/tools/build/+/1337600 which picked up the former's change to README.recipes.md.

Could we ensure that each CL that affects README.recipes.md also regens it?

Comment 1 by iannu...@google.com, Nov 16 

whoops. I thought it did this. This is definitely a bug.

Comment 2 by estaab@google.com, Nov 20

Status: Available (was: Untriaged)
►

Sign in to add a comment