Make the ruff pre-commit hook fix issues when possible

[RayBB]

Proposal

scripts/expire_accounts.py:1:1: I001 [*] Import block is un-sorted or un-formatted
  |
1 | / from datetime import datetime, UTC
2 | | import web
3 | | 
4 | | 
5 | | def delete_old_links():
  | |_^ I001
6 |       for doc in web.ctx.site.store.values(type="account-link"):
7 |           expiry_date = datetime.strptime(doc["expires_on"], "%Y-%m-%dT%H:%M:%S.%f")
  |
  = help: Organize imports

Found 1 error.
[*] 1 fixable with the `--fix` option.

Can we add the --fix option to our import linting rule to fix some common errors?

~ Mek (in a Slack DM)

Justification

Breakdown

Requirements Checklist

• [ ]

Related files

Stakeholders

---

Instructions for Contributors

Please run these commands to ensure your repository is up to date before creating a new branch to work on this issue and each time after pushing code to Github, because the pre-commit bot may add commits to your PRs upstream.

[drona-gyawali]

Yes, successfully added and utilized the --fix option to address common linting errors. Initially, the imports in scripts/expire_accounts.py were flagged as unorganized. I configured ruff properly in pyproject.toml by using [tool.ruff.lint] for lint-specific rules. Afterward, I ran:

ruff check --fix scripts/expire_accounts.py

This resolved the issue by automatically organizing the imports. Additionally, we ensured the linting configurations in pyproject.toml followed best practices for clarity and maintainability.

This approach ensures smoother development workflows by reducing manual fixes and maintaining code quality consistently.