Skip to content

Update pythonnet to 3.0.1 to support Python 3.10#63355

Merged
garethgreenaway merged 3 commits into
saltstack:masterfrom
twangboy:update_pythonnet
Jan 4, 2023
Merged

Update pythonnet to 3.0.1 to support Python 3.10#63355
garethgreenaway merged 3 commits into
saltstack:masterfrom
twangboy:update_pythonnet

Conversation

@twangboy
Copy link
Copy Markdown
Contributor

What does this PR do?

Updates pythonnet to 3.0.1 to support python 3.10 on Windows

Merge requirements satisfied?

[NOTICE] Bug fixes or features added to Salt require tests.

Commits signed with GPG?

Yes

@twangboy twangboy requested a review from a team as a code owner December 21, 2022 18:10
@twangboy twangboy requested review from waynew and removed request for a team December 21, 2022 18:10
@twangboy twangboy added the Sulfur v3006.0 release code name and version label Dec 21, 2022
@twangboy twangboy added this to the Sulphur v3006.0 milestone Dec 21, 2022
@Ch3LL
Copy link
Copy Markdown
Contributor

Ch3LL commented Dec 22, 2022

pre-commit is failing

@s0undt3ch
Copy link
Copy Markdown
Contributor

re-run full all

@s0undt3ch
Copy link
Copy Markdown
Contributor

@twangboy the windows failures are valid and likely due to the new library version.

@twangboy
Copy link
Copy Markdown
Contributor Author

Yeah, pre-commit doesn't run on Windows so I have to do all this stuff by hand.

@s0undt3ch
Copy link
Copy Markdown
Contributor

Yeah, pre-commit doesn't run on Windows so I have to do all this stuff by hand.

The windows tests don't start with this new version. Can you have a look a fix that?

@s0undt3ch
Copy link
Copy Markdown
Contributor

Not today, of course 😁

@twangboy
Copy link
Copy Markdown
Contributor Author

Looks like the new version of pythonnet may have broken one of the grains

@Ch3LL Ch3LL mentioned this pull request Jan 3, 2023
3 tasks
@garethgreenaway garethgreenaway merged commit 689882d into saltstack:master Jan 4, 2023
@twangboy twangboy deleted the update_pythonnet branch March 23, 2023 15:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Sulfur v3006.0 release code name and version

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants