Generate requirements_* from manifests

[rohankapoorcom]

Description:

Generate requirements_* from manifests (if present). If not, fallback to the current approach of reading REQUIREMENTS from the module attribute. I disabled exploring the children of the homeassistant.components.* packages since that will just add a dependency (from the manifest) due to each of the python files in the package. Just having one for the top level package should be sufficient.

Related issue (if applicable): relates to #22700

Checklist:

• The code change is tested and works locally.
• Local tests pass with tox. Your PR cannot be merged unless tests pass
• There is no commented out code in this PR.

[robbiet480]

1000 points to @rohankapoorcom for taking the initiative on this without prompting!

[codecov]

Codecov Report

Merging #22718 into dev will decrease coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##              dev   #22718      +/-   ##
==========================================
- Coverage   93.83%   93.83%   -0.01%     
==========================================
  Files         449      448       -1     
  Lines       36584    36524      -60     
==========================================
- Hits        34330    34271      -59     
+ Misses       2254     2253       -1

Impacted Files	Coverage Δ
homeassistant/requirements.py

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update eadc1e0...8f06cc2. Read the comment docs.

[awarecan]

We should do following in same PR

• Remove all REQUIREMENTS from .py code
• Change Pull Request template
• Change dev doc (in dev doc repo of course, but link to here)

[rohankapoorcom]

Reading #22700, I got the feeling that each of those were intended to be separate tasks. If you think they shouldn't be, I'm happy to make the changes.

[balloob]

I suggest we make them new tasks.

[awarecan]

Your call. But the elif block need remove, we have manifest validator is running on CI to ensure all components has to have manifest.json

[rohankapoorcom]

Your call. But the elif block need remove, we have manifest validator is running on CI to ensure all components has to have manifest.json

It's still needed for scripts/auth. I'm breaking them out into different methods per @balloob's above comment: #22718 (comment)

[rohankapoorcom]

Changes pushed up.

[awarecan]

This order changed. Any chance we can keep it ordered?

[awarecan]

I just merged a new integration in, #22684. Please rebase.

[rohankapoorcom]

I'll look at ordering and will rebase in the evening. Just headed into the office.

[awarecan]

I rebased and fixed the error. But not fix order yet.

EDIT: order is done.

[rohankapoorcom]

Thanks for the assist @awarecan!

[rohankapoorcom]

Rebased again - @balloob @awarecan - are we good to go here?

[robbiet480]

If we aren't good, we'll fix it in post, lets get this show on the road