Skip to content

Conversation

@Yashvijain1234
Copy link

Fixes #4092

This PR adds a new documentation_url field to the PackageData model for improved consistency and visibility.
It updates the Cargo handler to map the documentation field directly to this new attribute instead of storing it in extra_data.
All relevant model tests and Cargo fixtures have been updated to reflect this change.

Copy link
Member

@AyanSinhaMahapatra AyanSinhaMahapatra left a comment

Choose a reason for hiding this comment

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

Thanks @Yashvijain1234 see comments for your consideration.

"bug_tracking_url": null,
"code_view_url": null,
"vcs_url": null,
"documentation_url": null,
Copy link
Member

Choose a reason for hiding this comment

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

There would a lot of other place where we have packages inthe output JSON where we need to regen, could you do a:

pytest -vvs tests/ --test-suite=validate -n 4 && SCANCODE_REGEN_TEST_FIXTURES=yes pytest -vvs tests/ --test-suite=validate --lf after configuring from your branch to regen all the tests which fail?

Signed-off-by: Yashvijain1234 <yashvijain1234@example.com>
Yashvijain1234 added 2 commits December 17, 2025 22:04
Signed-off-by: Yashvijain1234 <yashvijain1234@example.com>
Signed-off-by: Yashvijain1234 <yashvijain1234@example.com>
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.

Consider adding new documentation_url to Package model

2 participants