New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Improve default value handling and type conversion #437

Merged

wesen merged 2 commits into go-go-golems:main from wesen:task/convert-date-defaults-properly

Feb 18, 2025

Collaborator

wesen commented Feb 18, 2025

Enhances the parameter system's handling of default values:

Change CheckParameterDefaultValueValidity to return converted value instead of just
validating
Add proper type conversion for rename-regexp fields
Update GitHub Actions to latest versions

Key changes:

Default values are now properly converted to their target type during validation

Testing:

Added enhanced value comparison helper for test assertions

wesen added 2 commits

February 18, 2025 15:34


          ✨ 💩 Add coerce utility

ea19be5


          ✨ Convert defaults and then store them as such, if given as string (h…

e0196d9

…elps for dates)

wesen merged commit 4c0df1f into go-go-golems:main

2 checks passed

wesen deleted the task/convert-date-defaults-properly branch

February 18, 2025 20:54

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet