Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Build 308 #2391

Merged
merged 1 commit into from
Oct 12, 2024
Merged

Build 308 #2391

merged 1 commit into from
Oct 12, 2024

Conversation

mhammond
Copy link
Owner

No description provided.

@Avasam
Copy link
Collaborator

Avasam commented Oct 12, 2024

The black failure is fixed either by bumping the min Python version to 3.9 for that CI job (like this https://github.com/mhammond/pywin32/pull/2207/files#diff-7829468e86c1cc5d5133195b5cb48e1ff6c75e3e9203777f6b2e379d9e4882b3R113-R115), pinning black to 24.8 instead of stable (I already pin other checkers anyway for that exact reason), or switching to Ruff format (#2389)

I'm working on adding a quick test for the postinstall script for sanity test, if you'd like that to be in first. (not 100% sure where it should go, but that can be discussed in the PR) #2392

@mhammond
Copy link
Owner Author

I think this is all we need for a new release - WDYT?

@mhammond mhammond requested a review from Avasam October 12, 2024 19:59
Copy link
Collaborator

@Avasam Avasam left a comment

Choose a reason for hiding this comment

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

Yeah looks like all the important recent bugfixes 👍

@mhammond mhammond merged commit 8578330 into main Oct 12, 2024
62 checks passed
@mhammond mhammond deleted the push-nzulumzsunlv branch October 12, 2024 20:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants