Add new integration for Jandy iAqualink pool control

[flz]

Description:

This is a new component introducing support for Jandy iAqualink pool controls.

This PR includes support for the climate platform only.

Pull request with documentation for home-assistant.io (if applicable): home-assistant/home-assistant.io#10165

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.
• I have followed the development checklist

If the code communicates with devices, web services, or third-party tools:

• The manifest file has all fields filled out correctly. Update and include derived files by running python3 -m script.hassfest.
• New or updated dependencies have been added to requirements_all.txt by running python3 -m script.gen_requirements_all.
• Untested files have been added to .coveragerc.

[homeassistant]

Hi @flz,

It seems you haven't yet signed a CLA. Please do so here.

Once you do that we will be able to review and accept this pull request.

Thanks!

[flz]

Ugh. This shouldn't try to push 56 commits.

[MartinHjelmare]

Thanks for the PR. Please start by removing the light, sensor and switch platforms from this PR. More platforms can come in later PRs, one platform per PR. We want PRs as small as possible for quicker merge.

[flz]

Thanks for the review.

Please update the documentation to reflect that new integrations shouldn't contain more than a single platform for the reasons you mention. I understand the goal of making PR somewhat self-contained but in the case of a new integration, this seems rather arbitrary, especially when undocumented.

[andrewsayre]

Please update the documentation to reflect that new integrations shouldn't contain more than a single platform for the reasons you mention.

The docs do indicate that, but please let us know how we could make it more clear -- or better yet, submit a PR to the docs themselves with your improvements. :)

[flz]

Apologies, the documentation is clear. I obviously have missed it!

[MartinHjelmare]

Please don't squash commits after review has started to make it easier for readers to track changes.

[MartinHjelmare]

Looks good!

[MartinHjelmare]

Can be merged when build passes.

[flz]

Neat, thanks again for the help!