Skip to content

Conversation

@TG1999
Copy link
Contributor

@TG1999 TG1999 commented Sep 1, 2022

Signed-off-by: Tushar Goel tushar.goel.dav@gmail.com

@TG1999 TG1999 force-pushed the add_namespace_filter branch from b7a140a to 36a73d0 Compare September 1, 2022 22:20
Signed-off-by: Tushar Goel <tushar.goel.dav@gmail.com>
@TG1999 TG1999 force-pushed the add_namespace_filter branch from 36a73d0 to 5d2a4c9 Compare September 1, 2022 22:21
@TG1999 TG1999 requested review from pombredanne and tdruez September 2, 2022 07:14
Copy link
Member

@pombredanne pombredanne left a comment

Choose a reason for hiding this comment

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

There is a just a minor nit for your review... I will merge as-is.. but let's make sure we always order fieds in the right purl field hierarchy

model = Package
fields = ["name", "type", "version", "subpath", "purl", "packagerelatedvulnerability__fix"]
fields = [
"name",
Copy link
Member

Choose a reason for hiding this comment

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

The list of fields should be ordered this way:

            "purl",
            "type",
            "namespace",
            "name",
            "version",
            "subpath",
            "packagerelatedvulnerability__fix",

@pombredanne pombredanne merged commit 25bea6b into aboutcode-org:main Sep 2, 2022
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