Updated sanitizeCategory to better account for category names.

Updated sanitizeCategory to better account for category names.

R=whesse@google.com

Committed: https://github.com/dart-lang/sdk/commit/9f9aafc3378fe15a9bb416f160ff7ed2b274e7cf

[mkroghj, 2017-09-01 09:36:14]

mkroghj@google.com changed reviewers:
+ whesse@chromium.org

[mkroghj, 2017-09-01 09:36:15]

ptal

[mkroghj, 2017-09-01 09:37:08]

mkroghj@google.com changed reviewers:
+ whesse@google.com
- whesse@chromium.org

[mkroghj, 2017-09-01 09:37:08]

ptal

[Bill Hesse, 2017-09-01 11:10:57]

lgtm

https://codereview.chromium.org/3011813002/diff/1/tools/gardening/lib/src/uti...
File tools/gardening/lib/src/util.dart (right):

https://codereview.chromium.org/3011813002/diff/1/tools/gardening/lib/src/uti...
tools/gardening/lib/src/util.dart:111: // Category name starts with a channel
number (normally consisting of 1-3
LGTM, but could you just regexp match $[0-9]+(.*)\|all^ and take the matching
group 1 in parentheses?

[mkroghj, 2017-09-01 11:37:10]

Description was changed from

==========
Updated sanitizeCategory to better account for category names.
==========

to

==========
Updated sanitizeCategory to better account for category names.

R=whesse@google.com

Committed:
https://github.com/dart-lang/sdk/commit/9f9aafc3378fe15a9bb416f160ff7ed2b274e7cf
==========

[mkroghj, 2017-09-01 11:37:12]

Committed patchset #2 (id:20001) manually as
9f9aafc3378fe15a9bb416f160ff7ed2b274e7cf (presubmit successful).

[mkroghj, 2017-09-01 11:37:34]

https://codereview.chromium.org/3011813002/diff/1/tools/gardening/lib/src/uti...
File tools/gardening/lib/src/util.dart (right):

https://codereview.chromium.org/3011813002/diff/1/tools/gardening/lib/src/uti...
tools/gardening/lib/src/util.dart:111: // Category name starts with a channel
number (normally consisting of 1-3
On 2017/09/01 11:10:57, Bill Hesse wrote:
> LGTM, but could you just regexp match $[0-9]+(.*)\|all^ and take the matching
> group 1 in parentheses?

swapped $ and ^.

Done