Skip to content
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.

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

fix: config is optional #2813

Merged
merged 1 commit into from
Feb 25, 2025
Merged

fix: config is optional #2813

merged 1 commit into from
Feb 25, 2025

Conversation

abraemer
Copy link
Contributor

Summary of changes

When loading an input file without config field, we replace the resulting undefined with an object that represents an empty config properly.

Context and reason for change

Since config is optional, we need to handle the case when it is not present in the opossum file

How can the changes be tested

Not directly since nothing uses getClassifications right now. However my next PR cause errors on opossum files without config section.

@mstykow mstykow self-assigned this Feb 24, 2025
@abraemer abraemer merged commit bff2ccb into main Feb 25, 2025
5 checks passed
@abraemer abraemer deleted the fix-make-config-optional branch February 25, 2025 10:47
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