Issue metadata
Sign in to add a comment
|
Spacey didn't find a trailing space in a CL in infra repo |
||||||||||||||||||||||
Issue descriptionIt looks like Spacey missed a trailing space. Location: https://chromium-review.googlesource.com/c/infra/infra/+/1086089/4/go/src/infra/tricium/api/v1/tricium.proto#54 Spacey did run on that patchset: https://chromium-swarm.appspot.com/task?id=3de5b2168777ed10&refresh=10&show_raw=1 https://tricium-dev.appspot.com/run/5656182857400320 Analysis of this specific patchset should be reproduceible with { "project": "infra", "paths": ["go/src/infra/tricium/api/v1/tricium.proto"] "gitCommit": { "url": "https://chromium.googlesource.com/infra/infra", "ref": "refs/changes/89/1086089/4" } } https://tricium-dev.appspot.com/rpcexplorer/services/tricium.Tricium/Analyze
,
Jun 5 2018
And the result was that it found the space: https://chromium-swarm.appspot.com/task?id=3de9cfcd2fe48310 So, it's still a mystery why it didn't find the spacey in the initial run. I ran it again on the gerrit change: { "project": "infra", "paths": [ "go/src/infra/tricium/api/v1/tricium.proto" ], "gerritRevision": { "host": "chromium-review.googlesoure.com", "project": "infra/infra", "change": "infra/infra~master~I2d99e3e8debe0f67f6dbf62e001daeb3e44d9308", "gitUrl": "https://chromium.googlesource.com/infra/infra", "gitRef": "refs/changes/89/1086089/4" } } And the result seemed to be that it found the space: https://tricium-dev.appspot.com/run/6283291569160192
,
Jun 8 2018
Closing since we don't know whether this is a recurring actual problem. We'll just wait until we perhaps see this again. |
|||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||
Comment 1 by qyears...@chromium.org
, Jun 5 2018