Skip to content
This repository has been archived by the owner on Apr 26, 2024. It is now read-only.

fix null conversion error for defaults #114

Merged
merged 1 commit into from
Oct 16, 2023

Conversation

nvnieuwk
Copy link
Collaborator

Fixes #85

Copy link
Collaborator

@mirpedrol mirpedrol left a comment

Choose a reason for hiding this comment

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

Left only one small comment, otherwise LGTM :)

@@ -61,6 +61,6 @@
"default": "itDoesExist"
}
},
"required": ["field_4", "field_5", "field_6"]
Copy link
Collaborator

Choose a reason for hiding this comment

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

Why do we need this change?

@nvnieuwk nvnieuwk merged commit e737858 into nextflow-io:master Oct 16, 2023
3 checks passed
@nvnieuwk nvnieuwk deleted the fix/null-conversion branch October 16, 2023 12:42
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
2 participants