Skip to content

gh-129158: fix CI lint post-merge gh-129160 #129180

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

Closed
wants to merge 1 commit into from

Conversation

picnixz
Copy link
Member

@picnixz picnixz commented Jan 22, 2025

@bedevere-app bedevere-app bot added awaiting core review tests Tests in the Lib/test dir labels Jan 22, 2025
@picnixz picnixz changed the title CI: fix lint after a9f5edbf5fb141ad172978b25483342125184ed2 gh-129158: fix lint after a9f5edbf5fb141ad172978b25483342125184ed2 Jan 22, 2025
@picnixz picnixz changed the title gh-129158: fix lint after a9f5edbf5fb141ad172978b25483342125184ed2 gh-129158: fix CI lint Jan 22, 2025
@picnixz
Copy link
Member Author

picnixz commented Jan 22, 2025

@vstinner I'll add the commit that broke the CI linter in the commit message instead of the title.

Copy link
Member

@vstinner vstinner left a comment

Choose a reason for hiding this comment

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

LGTM

@picnixz
Copy link
Member Author

picnixz commented Jan 22, 2025

Since #129181 has a better title, I'll close mine.

@picnixz
Copy link
Member Author

picnixz commented Jan 22, 2025

Ok. Let's be in sync...

@picnixz picnixz closed this Jan 22, 2025
@picnixz picnixz deleted the fix/ci/lint-129158 branch January 22, 2025 13:34
@picnixz
Copy link
Member Author

picnixz commented Jan 22, 2025

Closing in favor of Łukasz's PR since he has a better title!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
awaiting merge skip news tests Tests in the Lib/test dir
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants