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.

Sign up for GitHub

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

Jump to bottom

[3.11] GH-109408: Move the Python file whitespace check from patchcheck to pre-commit (GH-109891) #110641

Closed
wants to merge 3 commits into from
Closed

[3.11] GH-109408: Move the Python file whitespace check from patchcheck to pre-commit (GH-109891) #110641

wants to merge 3 commits into from

Conversation

AA-Turner
Copy link
Member

@AA-Turner AA-Turner commented Oct 10, 2023 •
edited by bedevere-app bot
Loading

(cherry picked from commit 08ec4a1)

@AA-Turner
[3.11] pythonGH-109408: Move the Python file whitespace check from pa…
26ccc48 
…tchcheck to pre-commit (pythonGH-109891)

Co-authored-by: Hugo van Kemenade <[email protected]>.
(cherry picked from commit 08ec4a1)

Co-authored-by: Adam Turner <[email protected]>
@bedevere-app bedevere-app bot mentioned this pull request Oct 10, 2023
Merged
@bedevere-app bedevere-app bot mentioned this pull request Oct 10, 2023
Closed
@AA-Turner AA-Turner mentioned this pull request Oct 10, 2023
Closed
hugovk
hugovk requested changes Oct 10, 2023
View reviewed changes
Copy link
Member

@hugovk hugovk left a comment

Choose a reason for hiding this comment

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

(Approved too soon...)

@AlexWaygood
Copy link
Member

Thanks! Closing for now, though, since the PR to main was reverted

@AA-Turner AA-Turner deleted the backport-08ec4a1-3.11 branch January 13, 2024 20:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@hugovk hugovk hugovk requested changes

@AlexWaygood AlexWaygood Awaiting requested review from AlexWaygood AlexWaygood is a code owner

Assignees
No one assigned
Labels
awaiting core review skip news
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@AA-Turner @AlexWaygood @hugovk