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

Reimplement get installed packages #2988

Merged
merged 9 commits into from
Jun 12, 2022

Conversation

JonoYang
Copy link
Contributor

@JonoYang JonoYang commented Jun 9, 2022

This PR adds the function get_installed_packages to packagedcode/plugin_packages.py as a replacement to get_installed_packages from packagedcode/alpine.py, packagedcode/debian.py, and packagedcode/rpm.py

@JonoYang JonoYang requested a review from pombredanne June 9, 2022 17:25
@JonoYang JonoYang force-pushed the reimplement-get-installed-packages branch 2 times, most recently from 5117a37 to 0676fe5 Compare June 9, 2022 23:24
JonoYang and others added 9 commits June 10, 2022 14:44
Signed-off-by: Jono Yang <jyang@nexb.com>
    * Update function signatures for get_installed_packages

Signed-off-by: Jono Yang <jyang@nexb.com>
    * Modify run_scan to return a codebase
    * Create new PackageWithResources class

Signed-off-by: Jono Yang <jyang@nexb.com>
Signed-off-by: Jono Yang <jyang@nexb.com>
Signed-off-by: Jono Yang <jyang@nexb.com>
    * Override to_dict on PackageWithResources
    * Update expected test results

Signed-off-by: Jono Yang <jyang@nexb.com>
Signed-off-by: Jono Yang <jyang@nexb.com>
Signed-off-by: Jono Yang <jyang@nexb.com>
Signed-off-by: Philippe Ombredanne <pombredanne@nexb.com>
@JonoYang JonoYang force-pushed the reimplement-get-installed-packages branch from 6cf681d to f2fcc3b Compare June 10, 2022 21:44
@pombredanne
Copy link
Member

tests/packagedcode/test_pypi.py::TestPyPiEndtoEnd::test_package_scan_pypi_end_to_end is failing only in the CI.
I am merging anyway.

@pombredanne pombredanne merged commit 4895e0b into develop Jun 12, 2022
@pombredanne pombredanne deleted the reimplement-get-installed-packages branch June 12, 2022 22:10
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