fixed pre-commit docformatter error

[musaqlain]

What

Fixes the docformatter pre-commit hook and pins language_version: python3.12
for hooks affected by Python 3.14 AST removals on pre-commit.ci.

Why

docformatter (InvalidManifestError)

• The previous repo: local definition was replaced with the official repo URL
• v1.7.5 contains a docformatter-venv hook using the deprecated language: python_venv,
  which is rejected by pre-commit v4+, even if only the docformatter hook is used,
  pre-commit validates the entire manifest
• v1.7.7 removes the problematic hook from its manifest
• However, v1.7.7 still depends on untokenize, which uses ast.Constant.s,
  removed in Python 3.14, causing a build failure on pre-commit.ci
• Ref: Installation fails on Python 3.14 PyCQA/docformatter#327, docformatter fails to install as a pre-commit hook when running with python-3.14 PyCQA/docformatter#340

nbQA (ast.Str removal)

• In Python 3.14, ast.Str was removed
• nbQA's notebook processing depends on ast.Str, so it fails on Python 3.14
• On main, this was hidden by pre-commit.ci's cached environments, the nbQA
  environment was built on an older Python and reused across runs
  As you can in this image (this is from the last PR merged by Josh), the environment is actually using cached nbQA that is using 3.12 python version.

- Updating the docformatter config invalidated the cache, forcing a fresh rebuild on Python 3.14, which exposed the latent failure (as u cann see in the image, it is using python3.14, and eventually it get failed)

• Pinning language_version: python3.12 ensures both hooks build on a compatible Python

Changes

• docformatter: repo: local → repo: https://github.com/PyCQA/docformatter at v1.7.7
• docformatter: added language_version: python3.12 + additional_dependencies: [tomli]
• nbQA hooks: added language_version: python3.12 to both nbqa-ruff-format and nbqa-ruff
• Removed redundant local hook fields (name, entry, language, types)

Fixes #1266

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 86.92%. Comparing base (408e150) to head (60a3d96).
⚠️ Report is 8 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1378      +/-   ##
==========================================
+ Coverage   86.87%   86.92%   +0.04%     
==========================================
  Files          24       24              
  Lines        3064     3204     +140     
==========================================
+ Hits         2662     2785     +123     
- Misses        402      419      +17     

Flag	Coverage Δ
unittests	86.92% <ø> (+0.04%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[musaqlain]

this pr precedes all the recent PRs' who are in queue and are getting pre-commit.ci error during building.

cc @henrykironde

[henrykironde]

Could we clean this up and remove the comments?

[henrykironde]

Could we minimize the changes here? Let’s try using the previous config and just add language_version: python3.12

[musaqlain]

done, reverted to the previous repo: local config and only added language_version: python3.12

Also worth noting: docformatter released v1.7.8 yesterday with Python 3.14 support. what do you think, should we use it? if we use this new version then we could switch from repo: local to repo: https://github.com/PyCQA/docformatter at rev: v1.7.8 and remove the language_version pin for docformatter (nbQA pin would still be needed though).

[musaqlain]

update: I tested v1.7.8 on my fork and it reformats some existing files due to bug fixes in the new version (#335, #336). so upgrading to v1.7.8 would be better as a separate follow-up PR. for now, the minimal language_version: python3.12 pin should be good.

what do you think? 👀

[henrykironde]

Can we try to undo line 20 and 25 and see if it passes

[musaqlain]

Can we try to undo lines 20 and 25 and see if it passes?

Hello Henry, I already tested it without pinning the Python 3.12 version on lines 20 and 25, and it failed. This is because it then defaults to Python 3.14; since ast.Str was removed in 3.14, and nbQA depends on it, it triggers the following error:

nbQA failed to process docs/user_guide/examples/Bird_FineTuning.ipynb 
with exception "AttributeError("module 'ast' has no attribute 'Str'")"

The language_version: python3.12 pin is required for nbQA until they release a fix.

--
Previously, nbQA was running correctly because it was using a cached 3.12 version. However, once I updated .pre-commit-config.yaml, it triggered nbQA to run using Python 3.14, which is why it failed.

[henrykironde]

Thanks @musaqlain for these tests.