Skip to content

fix: handle empty ContainerType values in CSV to YAML conversion#165

Merged
SlavaVedernikov merged 2 commits intoSlavaVedernikov:masterfrom
mnadenysenko:164-fix-blank-container-type
Oct 10, 2025
Merged

fix: handle empty ContainerType values in CSV to YAML conversion#165
SlavaVedernikov merged 2 commits intoSlavaVedernikov:masterfrom
mnadenysenko:164-fix-blank-container-type

Conversation

@mnadenysenko
Copy link
Copy Markdown
Contributor

fix: handle empty ContainerType values in CSV to YAML conversion

  • Change null check to IsNullOrWhiteSpace in CsvToJObjectAaCWriter
  • Update YAML reader to properly handle empty/invalid ContainerType values
  • Default to ContainerType.None when value is empty or whitespace
  • Add warning logging for invalid ContainerType values

Fixes #164

…ToJObjectAaCWriter for improved validation and normalization
Copy link
Copy Markdown
Owner

@SlavaVedernikov SlavaVedernikov left a comment

Choose a reason for hiding this comment

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

Thanks

@SlavaVedernikov SlavaVedernikov merged commit ae41627 into SlavaVedernikov:master Oct 10, 2025
@mnadenysenko mnadenysenko deleted the 164-fix-blank-container-type branch October 17, 2025 12:05
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.

Empty ContainerType in CSV Converted to Invalid YAML Value

2 participants