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

Populate package and dependency attributes in inspect_packages #1180

Merged
merged 3 commits into from
May 6, 2024

Conversation

AyanSinhaMahapatra
Copy link
Member

@AyanSinhaMahapatra AyanSinhaMahapatra commented Apr 25, 2024

Properly populate dependency attributes like for_package, datasource_id and datafile_resource for DiscoveredDependencies found in inspect_packages pipeline.
Also populate datasource_ids and codebase_resources correctly for packages.
Here we only assign to the package the codebase resource of the package manifest where it was found, as assigning all other package resources would require a full package assembly, and thus out of scope for this pipeline.

Copy link
Member

@tdruez tdruez left a comment

Choose a reason for hiding this comment

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

Missing tests and changelog entry.

@AyanSinhaMahapatra AyanSinhaMahapatra changed the title Populate dependency attributes in inspect_packages Populate package and depenedency attributes in inspect_packages Apr 29, 2024
Properly populate dependency attributes like for_package, datasource_id
and datafile_resource for DiscoveredDependencies and codebase_resources
in DiscoveredPackages found in inspect_packages pipeline.

Signed-off-by: Ayan Sinha Mahapatra <ayansmahapatra@gmail.com>
scanpipe/pipes/__init__.py Outdated Show resolved Hide resolved
scanpipe/pipes/__init__.py Outdated Show resolved Hide resolved
@mjherzog mjherzog changed the title Populate package and depenedency attributes in inspect_packages Populate package and dependency attributes in inspect_packages May 2, 2024
Signed-off-by: Ayan Sinha Mahapatra <ayansmahapatra@gmail.com>
@tdruez tdruez merged commit 6aa0afc into main May 6, 2024
9 checks passed
@tdruez tdruez deleted the fix-inspect-packages branch May 6, 2024 07:11
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.

None yet

2 participants