Commit 4f91d030 authored by Joe Mason's avatar Joe Mason Committed by Commit Bot

Clearer warning when components in OWNERS files match multiple teams

R=robertocn@chromium.org

Bug: 1013608
Change-Id: Icf9fd7235a22d91adacc1b1595353e0060147175
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1893375Reviewed-by: default avatarRoberto Carrillo <robertocn@chromium.org>
Commit-Queue: Joe Mason <joenotcharles@chromium.org>
Cr-Commit-Position: refs/heads/master@{#711475}
parent c617d58b
...@@ -147,8 +147,8 @@ def validate_mappings(options, args): ...@@ -147,8 +147,8 @@ def validate_mappings(options, args):
warnings = '' warnings = ''
for component, teams in affected_component_to_teams.iteritems(): for component, teams in affected_component_to_teams.iteritems():
if len(teams) > 1: if len(teams) > 1:
warnings += '\nComponent %s will map to %s' % ( warnings += ('\nThe set of all OWNERS files with COMPONENT: %s list '
component, ', '.join(teams)) "multiple TEAM's: %s") % (component, ', '.join(teams))
if warnings: if warnings:
warnings = ('Are you sure these are correct? After landing this patch:%s' warnings = ('Are you sure these are correct? After landing this patch:%s'
% warnings) % warnings)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment