Skip to content

Comments

Improve YAML parsing#580

Open
ielis wants to merge 3 commits intodevelopfrom
improve_yaml_parsing
Open

Improve YAML parsing#580
ielis wants to merge 3 commits intodevelopfrom
improve_yaml_parsing

Conversation

@ielis
Copy link
Collaborator

@ielis ielis commented Jul 15, 2022

The YAML parser must handle missing/empty fields without throwing NPE.

The PR prevents NPEs and bundles examples folder containing two example YAML configurations as well as a small VCF file into the distribution ZIP.

ielis added 3 commits July 14, 2022 16:07
Signed-off-by: Daniel Danis <daniel.gordon.danis@protonmail.com>
Signed-off-by: Daniel Danis <daniel.gordon.danis@protonmail.com>
Signed-off-by: Daniel Danis <daniel.gordon.danis@protonmail.com>
@pnrobinson
Copy link
Collaborator

@ielis -- just seeing this -- is this stale?

@ielis
Copy link
Collaborator Author

ielis commented May 12, 2023

No, I think part of this code can be useful. I need to look into this and then I'll merge..

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.

2 participants