Core: Integration Manifestation

Created on 4 Apr 2019  路  7Comments  路  Source: home-assistant/core

In #22699 we have introduced manifests to all integrations. Now it's time to use them when manifests are available.

Manifest validator

  • [x] Script to validate that each integration has a manifest with the required fields and that fields are the correct types #22708

Codeowners

  • [x] Generate CODEOWNERS based on a core template + fetch from manifests #22705
  • [x] Mention code owners when label of integration that they own is added to an issue or PR

Requirements

  • [x] Generate requirements_* based on manifests #22718
  • [x] Remove REQUIREMENTS from all integrations. #23024
  • [ ] Deprecate REQUIREMENTS for custom components
  • [x] Install requirements from manifest (fallback to current REQUIREMENTS) #22717

Dependencies

  • [x] Load dependencies from manifest (fallback to current DEPENDENCIES) #22716
  • [x] Remove DEPENDENCIES from all integrations. #23024
  • [ ] Deprecate DEPENDENCIES for custom components
  • [x] Use manifest when determining circular dependencies #22908

Document update

  • [x] PR template #22751
  • [x] Dev doc update home-assistant/developers.home-assistant#214

Most helpful comment

363b3c5 馃憤

All 7 comments

I'm working on

Load dependencies from manifest (fallback to current DEPENDENCIES)
Install requirements from manifest (fallback to current REQUIREMENTS).
Generate requirements_* based on manifests

What is missing here? DEPENDENCIES and REQUIREMENTS are removed from integrations if I didn't miss any.

If it's possible, we should install integration requirements before validating the integration config so requirements are available in config validation.

I will add that to #23112

363b3c5 馃憤

I think this issue has been completed and can be closed @balloob ?

Yessssss 馃拑

Was this page helpful?
0 / 5 - 0 ratings

Related issues

aweb-01 picture aweb-01  路  3Comments

piitaya picture piitaya  路  3Comments

sibbl picture sibbl  路  3Comments

kirichkov picture kirichkov  路  3Comments

arangates picture arangates  路  3Comments