Skip to content

adding a check for choices-enabled runtime parameters - #164

Merged
keighrim merged 4 commits into
developfrom
160-fix-params-w-choices
Jun 30, 2023
Merged

adding a check for choices-enabled runtime parameters#164
keighrim merged 4 commits into
developfrom
160-fix-params-w-choices

Conversation

@keighrim

Copy link
Copy Markdown
Member

fixes#160.

@codecov

codecovBot commented Jun 26, 2023

Copy link
Copy Markdown

Codecov Report

Merging #164 (a54984c) into develop (a7ce71b) will increase coverage by 0.08%.
The diff coverage is 100.00%.

❗ Current head a54984c differs from pull request most recent head ffd1e13. Consider uploading reports for the commit ffd1e13 to get more accurate results

@@ Coverage Diff @@## develop #164 +/- ##
===========================================
+ Coverage 75.49% 75.57% +0.08% 
===========================================
Files 7 7 Lines 604 606 +2 ===========================================
+ Hits 456 458 +2 
Misses 148 148 
FlagCoverage Δ
unittests75.57% <100.00%> (+0.08%)⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted FilesCoverage Δ
clams/app/__init__.py87.28% <100.00%> (+0.21%)⬆️

@keighrim
keighrim merged commit e921b14 into developJun 30, 2023
@keighrim
keighrim deleted the 160-fix-params-w-choices branch June 30, 2023 21:46
@keighrimkeighrim mentioned this pull request Jul 19, 2023
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

no check for values of parameters when choices is given

1 participant

@keighrim