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

osrd_schemas: update to 'poetry:2.0' and use 'ruff' for linting #10724

Merged
merged 3 commits into from
Feb 13, 2025

Conversation

woshilapin
Copy link
Contributor

@woshilapin woshilapin commented Feb 7, 2025

2 commits because bumping to Python 3.12 was problematic with some of the linting tools. So first move to ruff, then move to pyright instead of pytype because pytype doesn't work on Python 3.12., then move all the project to the future (newer Python's version and modern pyproject.toml).

@woshilapin woshilapin requested review from a team as code owners February 7, 2025 14:39
@github-actions github-actions bot added area:railjson Work on Proposed Unified Rail Assets Data Exchange Format area:ci Work on Continous Integration Pipeline Service labels Feb 7, 2025
@codecov-commenter
Copy link

codecov-commenter commented Feb 7, 2025

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 81.93%. Comparing base (b443b6a) to head (45617c7).
Report is 55 commits behind head on dev.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@           Coverage Diff           @@
##              dev   #10724   +/-   ##
=======================================
  Coverage   81.93%   81.93%           
=======================================
  Files        1078     1078           
  Lines      107544   107587   +43     
  Branches      733      733           
=======================================
+ Hits        88115    88154   +39     
- Misses      19390    19394    +4     
  Partials       39       39           
Flag Coverage Δ
editoast 74.38% <ø> (-0.02%) ⬇️
front 89.42% <ø> (+<0.01%) ⬆️
gateway 2.18% <ø> (ø)
osrdyne 3.28% <ø> (ø)
railjson_generator 87.50% <ø> (ø)
tests 88.14% <ø> (ø)

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.

@woshilapin woshilapin force-pushed the wsl/schemas/poetry-2 branch 3 times, most recently from f719b41 to 28d785b Compare February 7, 2025 15:19
@woshilapin woshilapin force-pushed the wsl/schemas/poetry-2 branch from 28d785b to 04d54f8 Compare February 7, 2025 15:42
@woshilapin woshilapin force-pushed the wsl/schemas/poetry-2 branch 5 times, most recently from f18a334 to b80a1d0 Compare February 7, 2025 21:54
Signed-off-by: Jean SIMARD <[email protected]>
@woshilapin woshilapin force-pushed the wsl/schemas/poetry-2 branch 2 times, most recently from b9fc234 to 0a0cd6a Compare February 7, 2025 22:15
@woshilapin woshilapin force-pushed the wsl/schemas/poetry-2 branch from 0a0cd6a to 550500a Compare February 7, 2025 23:24
@github-actions github-actions bot added the area:front Work on Standard OSRD Interface modules label Feb 7, 2025
@woshilapin woshilapin enabled auto-merge February 11, 2025 09:35
@woshilapin woshilapin added this pull request to the merge queue Feb 12, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Feb 12, 2025
@Khoyo Khoyo added this pull request to the merge queue Feb 13, 2025
Merged via the queue into dev with commit da06215 Feb 13, 2025
27 checks passed
@Khoyo Khoyo deleted the wsl/schemas/poetry-2 branch February 13, 2025 08:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:ci Work on Continous Integration Pipeline Service area:front Work on Standard OSRD Interface modules area:railjson Work on Proposed Unified Rail Assets Data Exchange Format
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants