Update to fparser 0.2.4

[sergisiso]

No description provided.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (bb44e5d) to head (2718e5d).

Additional details and impacted files

@@            Coverage Diff            @@
##            master     #3440   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files          395       395           
  Lines        55199     55199           
=========================================
  Hits         55199     55199           

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

[sergisiso]

One pytest failed, I am a bit confused because adding the same fortran snippet into a file and doing: fparser2 test.f90 and psyclone test.f90 both succeed.

[arporter]

One pytest failed, I am a bit confused because adding the same fortran snippet into a file and doing: fparser2 test.f90 and psyclone test.f90 both succeed.

Could it be a Python version thing?

[sergisiso]

No, I am using the same environment for everything. The problem is that the test requests a fixed_form parser and the snippet is free_form. Maybe before we just ignored the syntaxerror because it starts with a blankline and we fixed this issue now.

This is not a problem in most cases because our FortranReader defaults to free_form True. But this particular tests uses the FparserReader that defaults to free_form False.

[sergisiso]

@arporter @LonelyCat124 This is ready for review

[arporter]

I suggest we pause this while we get 0.2.4 of fparser out.

[sergisiso]

@arporter I unblocked the PR and marked ready for review again.

[arporter]

Thanks Sergi, you just need to put the GHA workflow back to using pypi and I have a question about why other changes in that file are showing.

[sergisiso]

@arporter ready for another look

[arporter]

All good now. I'll double-check the integration tests since we've updated the workflow files.

[arporter]

ITs were all green so I'll proceed to merge.