From 8bebd8583f78ce3a829b7f7517b9181f7ef35e5c Mon Sep 17 00:00:00 2001 From: cdce8p <30130371+cdce8p@users.noreply.github.com> Date: Mon, 8 Apr 2019 06:01:05 +0200 Subject: [PATCH] Fix manifest codeowners (#22871) * Added individual files section * Replaced some manifest/codeowners --- CODEOWNERS | 8 ++++++-- homeassistant/components/cover/manifest.json | 2 +- homeassistant/components/demo/manifest.json | 2 +- script/manifest/codeowners.py | 8 ++++++++ 4 files changed, 16 insertions(+), 4 deletions(-) diff --git a/CODEOWNERS b/CODEOWNERS index a5c06f991c9..6ce7388e0d1 100644 --- a/CODEOWNERS +++ b/CODEOWNERS @@ -45,13 +45,13 @@ homeassistant/components/configurator/* @home-assistant/core homeassistant/components/conversation/* @home-assistant/core homeassistant/components/coolmaster/* @OnFreund homeassistant/components/counter/* @fabaff -homeassistant/components/cover/* @cdce8p +homeassistant/components/cover/* @home-assistant/core homeassistant/components/cpuspeed/* @fabaff homeassistant/components/cups/* @fabaff homeassistant/components/daikin/* @fredrike @rofrantz homeassistant/components/darksky/* @fabaff homeassistant/components/deconz/* @kane610 -homeassistant/components/demo/* @fabaff +homeassistant/components/demo/* @home-assistant/core homeassistant/components/digital_ocean/* @fabaff homeassistant/components/discogs/* @thibmaek homeassistant/components/doorbird/* @oblogic7 @@ -243,3 +243,7 @@ homeassistant/components/zha/* @dmulcahey @adminiuga homeassistant/components/zone/* @home-assistant/core homeassistant/components/zoneminder/* @rohankapoorcom homeassistant/components/zwave/* @home-assistant/z-wave + +# Individual files +homeassistant/components/group/cover @cdce8p +homeassistant/components/demo/weather @fabaff diff --git a/homeassistant/components/cover/manifest.json b/homeassistant/components/cover/manifest.json index f39f7fb0650..da5a644334c 100644 --- a/homeassistant/components/cover/manifest.json +++ b/homeassistant/components/cover/manifest.json @@ -7,6 +7,6 @@ "group" ], "codeowners": [ - "@cdce8p" + "@home-assistant/core" ] } diff --git a/homeassistant/components/demo/manifest.json b/homeassistant/components/demo/manifest.json index 08cf75a3c53..859cac597dc 100644 --- a/homeassistant/components/demo/manifest.json +++ b/homeassistant/components/demo/manifest.json @@ -9,6 +9,6 @@ "zone" ], "codeowners": [ - "@fabaff" + "@home-assistant/core" ] } diff --git a/script/manifest/codeowners.py b/script/manifest/codeowners.py index 9745f3b82f2..96b2b252e3d 100755 --- a/script/manifest/codeowners.py +++ b/script/manifest/codeowners.py @@ -27,6 +27,12 @@ homeassistant/scripts/check_config.py @kellerza # Integrations """ +INDIVIDUAL_FILES = """ +# Individual files +homeassistant/components/group/cover @cdce8p +homeassistant/components/demo/weather @fabaff +""" + def generate(): """Generate CODEOWNERS.""" @@ -39,6 +45,8 @@ def generate(): parts.append("homeassistant/components/{}/* {}".format( manifest['domain'], ' '.join(manifest['codeowners']))) + parts.append('\n' + INDIVIDUAL_FILES.strip()) + return '\n'.join(parts)