Ruff 0.15.10

[armanddidierjean]

Description

Summary

Ruff 0.15.10 compatibility

Required PRs

Requires #985

Classification

Type of Change

• 🐛 Bug fix (non-breaking change which fixes an issue)
• ✨ New feature (non-breaking change which adds functionality)
• 🔨 Refactor (non-breaking change that neither fixes a bug nor adds a feature)
• 🔧 Infra CI/CD (changes to configs of workflows)
• 💥 BREAKING CHANGE (fix or feature that require a new minimal version of the front-end)
• 😶‍🌫️ No impact for the end-users

Impact & Scope

• Core functionality changes
• Single module changes
• Multiple modules changes
• Database migrations required
• Other: ...

Testing

• 1. Tested this locally
• 2. Added/modified tests that pass the CI (or tested in a downstream fork)
• 3. Tested in a deployed pre-prod
• 0. Untestable (exceptionally), will be tested in prod directly

Documentation

• Updated the docs accordingly :
• " Docstrings
• # Inline comments
• No documentation needed

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 84.42%. Comparing base (764a6ab) to head (24e76bc).

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #986      +/-   ##
==========================================
- Coverage   84.43%   84.42%   -0.01%     
==========================================
  Files         206      206              
  Lines       14956    14948       -8     
==========================================
- Hits        12628    12620       -8     
  Misses       2328     2328              

☔ View full report in Codecov by Harness.
📢 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.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[Marc-Andrieu]

I don't get the removal of target-version = "py314" in the pyproject.toml and the r"" in the test utils... that's all

As @Luhmos promised, that was actually a rapid review! Good job

[armanddidierjean]

I don't get the removal of target-version = "py314" @Marc-Andrieu

It was in contradiction with "requires-python = ">= 3.12, < 3.15": we allowed for Python 3.12 but asked Ruff to make changes targeting 3.14

Ruff documentation also recommand the usage of project.requires-python instead of target-version

[armanddidierjean]

the r"" in the test utils @Marc-Andrieu

pytest.raises's match argument ask for a regex, the good practice seems to be the usage of a raw string. This change was done automatically by ruff