Fix icc check - Githubissues

A-CGray commented 4 months ago

Purpose

Inline comment messes with the ifdef check, see https://github.com/mdolab/pyhyp/pull/87#issuecomment-2115027287 for details

Expected time until merged

Type of change

[ ] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (non-backwards-compatible fix or feature)
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no API changes)
[ ] Documentation update
[ ] Maintenance update
[ ] Other (please describe)

Testing

Checklist

[ ] I have run flake8 and black to make sure the Python code adheres to PEP-8 and is consistently formatted
[ ] I have formatted the Fortran code with fprettify or C/C++ code with clang-format as applicable
[ ] I have run unit and regression tests which pass locally with my changes
[ ] I have added new tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation

codecov[bot] commented 4 months ago

Codecov Report

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

Project coverage is 73.42%. Comparing base (6e85c53) to head (9d52739). Report is 1 commits behind head on main.

Additional details and impacted files

```diff @@ Coverage Diff @@ ## main #97 +/- ## ======================================= Coverage 73.42% 73.42% ======================================= Files 6 6 Lines 760 760 ======================================= Hits 558 558 Misses 202 202 ```

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

awccopp commented 4 months ago

Looks good to me

mdolab / idwarp