CycloneDX / CycloneDX/cyclonedx-python-lib

move `model.bom.Bom.validate()` to `validation.models`

Open
#455 4 comments 0 reactions 0 assignees Claimed by @saquibsaifee View on GitHub
breaking change help wanted
Dominant language
Python
Stars
116
Forks
67
Avg merge
8d 2h
Merged PRs (30d)
2

Description

now that a package `validation` was established via https://github.com/CycloneDX/cyclonedx-python-lib/issues/432,
t should be possible to move the data model validation there, too.\

data model validation is getting more and more complex https://github.com/CycloneDX/cyclonedx-python-lib/issues/453
and should be encapsulated and could be devided.

Contributor guide

Open the contributing guide

Research direction

Check open pull request #935 before starting, then inspect model.bom.Bom.validate() and the validation.models package established in issue #432. Review issue #453 for the growing validation scope; done means the data model validation is encapsulated in validation.models rather than remaining on Bom.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
data
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.