Skip to content

Fix adding parameter definition with parameter_type_list set to None#619

Merged
soininen merged 1 commit intomasterfrom
fix_add_parameter_definition
Mar 24, 2026
Merged

Fix adding parameter definition with parameter_type_list set to None#619
soininen merged 1 commit intomasterfrom
fix_add_parameter_definition

Conversation

@soininen
Copy link
Copy Markdown
Contributor

@soininen soininen commented Mar 24, 2026

This fixes a Traceback when parameter definition was added with parameter_type_list set to None.

No associated issue.

Checklist before merging

  • Documentation (also in Toolbox repo) is up-to-date
  • Release notes have been updated
  • Unit tests have been added/updated accordingly
  • Code has been formatted by black & isort
  • Unit tests pass

Parameter definition's parameter_type_list is optional, so it can
be None.
@codecov
Copy link
Copy Markdown

codecov bot commented Mar 24, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 86.15%. Comparing base (82d214b) to head (7a030cd).
⚠️ Report is 2 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #619   +/-   ##
=======================================
  Coverage   86.15%   86.15%           
=======================================
  Files          83       83           
  Lines       11475    11475           
  Branches     1636     1636           
=======================================
  Hits         9886     9886           
  Misses       1238     1238           
  Partials      351      351           

☔ View full report in Codecov by Sentry.
📢 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.

@soininen soininen merged commit e0d1efd into master Mar 24, 2026
17 checks passed
@soininen soininen deleted the fix_add_parameter_definition branch March 24, 2026 11:03
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.

1 participant