search

stfc / fparser

This project maintains and develops a Fortran parser called fparser2 written purely in Python which supports Fortran 2003 and some Fortran 2008. A legacy parser fparser1 is also available but is not supported. The parsers were originally part of the f2py project by Pearu Peterson.
https://fparser.readthedocs.io
Other
64 stars 29 forks source link

fix type guard stmt bug and update tests. #438

Closed rupertford closed 4 months ago

rupertford commented 11 months ago

There was a bug in the type_guard_stmt match method. Updated the code and associated tests. The restructuring of the code actually removes the offending line of code.

codecov[bot] commented 11 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (f43a18f) 91.96% compared to head (7ea0f86) 92.02%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #438 +/- ## ========================================== + Coverage 91.96% 92.02% +0.05% ========================================== Files 85 85 Lines 13644 13643 -1 ========================================== + Hits 12548 12555 +7 + Misses 1096 1088 -8 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

arporter commented 7 months ago

I'll take this on - there's not much to do.

codecov-commenter commented 7 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 92.13%. Comparing base (5f24ee2) to head (41a0aa4).

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #438 +/- ## ========================================== + Coverage 92.07% 92.13% +0.05% ========================================== Files 85 85 Lines 13825 13824 -1 ========================================== + Hits 12730 12737 +7 + Misses 1095 1087 -8 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

arporter commented 7 months ago

Just finishes off a small bugfix. Ready for review from either @hiker or @sergisiso.