Skip to content

Replace merge statements for optional parameters#202

Merged
awvwgk merged 4 commits intodftd4:mainfrom
marvinfriede:nag-fix
May 14, 2023
Merged

Replace merge statements for optional parameters#202
awvwgk merged 4 commits intodftd4:mainfrom
marvinfriede:nag-fix

Conversation

@marvinfriede
Copy link
Copy Markdown
Member

Applying the suggestions from #201.

@codecov
Copy link
Copy Markdown

codecov bot commented May 4, 2023

Codecov Report

Merging #202 (fae6917) into main (bd278d4) will increase coverage by 0.05%.
The diff coverage is 43.75%.

@@            Coverage Diff             @@
##             main     #202      +/-   ##
==========================================
+ Coverage   42.81%   42.87%   +0.05%     
==========================================
  Files          31       31              
  Lines        2639     2652      +13     
  Branches     1095     1096       +1     
==========================================
+ Hits         1130     1137       +7     
- Misses        971      977       +6     
  Partials      538      538              
Impacted Files Coverage Δ
src/dftd4/param.f90 38.88% <43.75%> (+0.66%) ⬆️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

Co-authored-by: Eisuke Kawashima <[email protected]>
@marvinfriede marvinfriede requested a review from awvwgk May 14, 2023 09:05
@awvwgk awvwgk merged commit 75d502e into dftd4:main May 14, 2023
@marvinfriede marvinfriede deleted the nag-fix branch May 14, 2023 12:23
@marvinfriede marvinfriede linked an issue May 14, 2023 that may be closed by this pull request
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.

Optional argument used before checking present

3 participants