Skip to content

Comments

Default to python 3.10#169

Closed
CoMPaTech wants to merge 8 commits intomainfrom
py310
Closed

Default to python 3.10#169
CoMPaTech wants to merge 8 commits intomainfrom
py310

Conversation

@CoMPaTech
Copy link
Member

No description provided.

@CoMPaTech CoMPaTech added the enhancement New feature or request label Mar 22, 2022
@CoMPaTech CoMPaTech requested a review from bouwew March 22, 2022 08:25
@CoMPaTech CoMPaTech self-assigned this Mar 22, 2022
CACHE_VERSION: 6
DEFAULT_PYTHON: "3.9"
CACHE_VERSION: 7
DEFAULT_PYTHON: "3.10"
Copy link
Contributor

@frenck frenck Mar 22, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I won't recommend that, as this allows for sneaking in 3.10 features, that break 3.9 unexpectedly.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good call, just as much I'm not sure why it's breaking at black on 3.9 references now ... I'll look at it again with a fresh pair of eyes sometime later today.

needs: prepare-test-cache
strategy:
matrix:
python-version: [3.9, "3.10"]
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is the library removing 3.9 support?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not intended as such

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just wanted to move default to 3.10 and still support 3.9 - as it's mainly focussed at HA we wouldn't drop until Core drops support

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Its a matrix, there is no default :)

@CoMPaTech CoMPaTech marked this pull request as draft March 22, 2022 08:56
Copy link
Contributor

@bouwew bouwew left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@CoMPaTech CoMPaTech closed this Mar 22, 2022
@CoMPaTech CoMPaTech deleted the py310 branch March 22, 2022 14:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants