Skip to content

(Closes #310) allow for current_scope to be None and extend tests#311

Merged
rupertford merged 5 commits intomasterfrom
310_allow_no_scope
Mar 15, 2022
Merged

(Closes #310) allow for current_scope to be None and extend tests#311
rupertford merged 5 commits intomasterfrom
310_allow_no_scope

Conversation

@arporter
Copy link
Member

@arporter arporter commented Feb 8, 2022

No description provided.

@arporter arporter self-assigned this Feb 8, 2022
@arporter arporter added the bug label Feb 8, 2022
@codecov
Copy link

codecov bot commented Feb 8, 2022

Codecov Report

Merging #311 (03ac629) into master (e0b14f8) will increase coverage by 0.02%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #311      +/-   ##
==========================================
+ Coverage   91.09%   91.11%   +0.02%     
==========================================
  Files          36       36              
  Lines       13107    13108       +1     
==========================================
+ Hits        11940    11944       +4     
+ Misses       1167     1164       -3     
Impacted Files Coverage Δ
src/fparser/two/Fortran2003.py 92.99% <100.00%> (+0.06%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e0b14f8...03ac629. Read the comment docs.

@arporter
Copy link
Member Author

arporter commented Feb 8, 2022

Ready for review from either @sergisiso or @rupertford.

Copy link
Collaborator

@rupertford rupertford left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice job. I've just requested a few more tests for the changes to Fortran2003.py

@rupertford rupertford added reviewed with actions PR has been reviewed and is back with developer and removed under review labels Feb 9, 2022
@arporter arporter added in progress and removed reviewed with actions PR has been reviewed and is back with developer labels Feb 10, 2022
@arporter
Copy link
Member Author

Ready for another look now.

Copy link
Collaborator

@rupertford rupertford left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just the one issue.

@rupertford rupertford added reviewed with actions PR has been reviewed and is back with developer and removed under review labels Mar 15, 2022
@arporter
Copy link
Member Author

Ready for another look now @rupertford.

@arporter arporter added ready for review and removed reviewed with actions PR has been reviewed and is back with developer labels Mar 15, 2022
Copy link
Collaborator

@rupertford rupertford left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All looks good. Approving.

@rupertford rupertford added ready for merge PR is waiting on final CI checks before being merged. and removed under review labels Mar 15, 2022
@rupertford rupertford merged commit c44c53c into master Mar 15, 2022
@rupertford rupertford deleted the 310_allow_no_scope branch March 15, 2022 17:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug ready for merge PR is waiting on final CI checks before being merged.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants