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

Update precommit monthly python 3.7+ #7542

Merged
merged 1 commit into from
Apr 22, 2022
Merged

Update precommit monthly python 3.7+ #7542

merged 1 commit into from
Apr 22, 2022

Conversation

glenn-jocher
Copy link
Member

@glenn-jocher glenn-jocher commented Apr 22, 2022

πŸ› οΈ PR Summary

Made with ❀️ by Ultralytics Actions

🌟 Summary

Improving code quality by updating pre-commit hooks and scheduling.

πŸ“Š Key Changes

  • Changed the pre-commit autoupdate schedule from quarterly to monthly.
  • Upgraded Python code formatting to apply to version 3.7 and newer (--py37-plus).
  • Tweaked Markdown (MD) formatting exclusion pattern.

🎯 Purpose & Impact

  • πŸ”„ More Frequent Updates: Monthly updates to pre-commit hooks will ensure code standards are regularly checked with the latest tools, providing more consistent quality over time.
  • 🐍 Python Version Upgrade: The shift to --py37-plus indicates a move towards newer Python features and away from deprecated 3.6 syntax, which can lead to more optimized and future-proof code.
  • ✏️ Markdown Formatting: The modification in the MD formatting exclusion pattern might mean increased consistency in documentation with fewer exclusions to formatting rules.

Users can expect better-maintained code with adherence to the latest best practices in code formatting and a gradual phasing out of older Python syntax.

@glenn-jocher glenn-jocher self-assigned this Apr 22, 2022
@glenn-jocher glenn-jocher merged commit e04d8b1 into master Apr 22, 2022
@glenn-jocher glenn-jocher deleted the update/precommit branch April 22, 2022 20:43
BjarneKuehl pushed a commit to fhkiel-mlaip/yolov5 that referenced this pull request Aug 26, 2022
ctjanuhowski pushed a commit to ctjanuhowski/yolov5 that referenced this pull request Sep 8, 2022
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.

None yet

1 participant