Skip to content

[python] remove fortran#288

Open
tmi wants to merge 1 commit into
developfrom
python/removeFortran
Open

[python] remove fortran#288
tmi wants to merge 1 commit into
developfrom
python/removeFortran

Conversation

@tmi

@tmi tmi commented Jul 17, 2026

Copy link
Copy Markdown
Contributor

disable fortran

disable mulio manage

remove fckit as a dependency

none of that is needed in the wheels anymore

Description

Contributor Declaration

By opening this pull request, I affirm the following:

  • All authors agree to the Contributor License Agreement.
  • The code follows the project's coding standards.
  • I have performed self-review and added comments where needed.
  • I have added or updated tests to verify that my changes are effective and functional.
  • I have run all existing tests and confirmed they pass.

🌈🌦️📖🚧 Documentation 🚧📖🌦️🌈
https://sites.ecmwf.int/docs/dev-section/multio/pull-requests/PR-288

@tmi
tmi force-pushed the python/removeFortran branch from a7c4321 to 41fead7 Compare July 17, 2026 13:00
@tmi
tmi marked this pull request as ready for review July 17, 2026 14:05
@tmi
tmi force-pushed the python/removeFortran branch from 41fead7 to 9361064 Compare July 17, 2026 14:06
@tmi
tmi requested a review from MircoValentiniECMWF July 17, 2026 14:06
@MircoValentiniECMWF

Copy link
Copy Markdown
Collaborator

I need to run a few checks before merging. The commit itself looks fine, but the failures above appear to be related to features that were removed a long time ago.

disable fortran

disable mulio manage

remove fckit as a dependency

none of that is needed in the wheels anymore
@dsarmany
dsarmany force-pushed the python/removeFortran branch from 9361064 to 747b87c Compare July 17, 2026 17:16
@codecov-commenter

codecov-commenter commented Jul 17, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 33.51%. Comparing base (3395b02) to head (747b87c).

Additional details and impacted files
@@           Coverage Diff            @@
##           develop     #288   +/-   ##
========================================
  Coverage    33.51%   33.51%           
========================================
  Files          379      379           
  Lines        42648    42648           
  Branches      1839     1839           
========================================
  Hits         14292    14292           
  Misses       28356    28356           

☔ 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.

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.

3 participants