Skip to content

Add fortitude linter - #64

Merged
awvwgk merged 1 commit into
grimme-lab:mainfrom
awvwgk:fortitude
Jul 20, 2026
Merged

Add fortitude linter#64
awvwgk merged 1 commit into
grimme-lab:mainfrom
awvwgk:fortitude

Conversation

@awvwgk

@awvwgk awvwgk commented Jul 19, 2026

Copy link
Copy Markdown
Member

Main changes

  • alphabetically sorted use statements
  • change single with double quotes
  • exclude lapack and blas interface for assumed size arrays
  • add intent input variable in main prog
  • propagate dealloc stat in lapack interface
  • make use statement for wsc explicit with only list

@codecov

codecov Bot commented Jul 19, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 14.28571% with 6 lines in your changes missing coverage. Please review.
✅ Project coverage is 30.68%. Comparing base (282626e) to head (2ed1b7c).
⚠️ Report is 23 commits behind head on main.

Files with missing lines Patch % Lines
app/main.f90 16.66% 5 Missing ⚠️
test/unit/main.f90 0.00% 1 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##             main      #64       +/-   ##
===========================================
- Coverage   51.57%   30.68%   -20.90%     
===========================================
  Files          18        2       -16     
  Lines        1648      176     -1472     
  Branches      746       72      -674     
===========================================
- Hits          850       54      -796     
+ Misses        390       85      -305     
+ Partials      408       37      -371     

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

@awvwgk
awvwgk merged commit 0642d32 into grimme-lab:main Jul 20, 2026
8 of 10 checks passed
@awvwgk
awvwgk deleted the fortitude branch July 20, 2026 10:45
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.

1 participant