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

pythonlib: Add missing docstring summary lines to fix D402 errors #5378

Merged
merged 1 commit into from
Mar 13, 2025

Conversation

arohanajit
Copy link
Contributor

This error popped up post updating ruff to latest version

@github-actions github-actions bot added Python Related code is in Python libraries labels Mar 13, 2025
Copy link
Member

@echoix echoix left a comment

Choose a reason for hiding this comment

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

Thanks! It would have appeared this weekend indeed. They released a minor (aka major) version today, with breaking changes and new stabilized rules.

Since I'm probably not available this weekend, at least, not on my computer, do you want to take a look at what is needed to do to have the version change go through correctly? I'll trigger the PR so you can look at what's missing after this -> #5381

@echoix echoix merged commit e9bb524 into OSGeo:main Mar 13, 2025
28 checks passed
@github-actions github-actions bot added this to the 8.5.0 milestone Mar 13, 2025
@arohanajit
Copy link
Contributor Author

Thanks! It would have appeared this weekend indeed. They released a minor (aka major) version today, with breaking changes and new stabilized rules.

Since I'm probably not available this weekend, at least, not on my computer, do you want to take a look at what is needed to do to have the version change go through correctly? I'll trigger the PR so you can look at what's missing after this -> #5381

Yeah, I have updated to newer version via pip and will put PRs if any of them breaks our existing checks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
libraries Python Related code is in Python
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants