nexB / python-inspector

Inspect Python code and PyPI package manifests. Resolve Python dependencies.
20 stars 17 forks source link

Added option to get resolved packages in a requirements.txt file. Fixes #135 #160

Open arijitde92 opened 8 months ago

arijitde92 commented 8 months ago

Created a new option --resolved-output that finds resolved packages and writes those into the given filename in a typical requirements.txt file format. Also added test test_resolved_cli in test_cli.py.

arijitde92 commented 8 months ago

I was having an issue rebasing in the previous PR #156 . Hence opened this new PR. It is working as expected. Please check.

TG1999 commented 8 months ago

@arijitde92 tests are failing in CI, please check. Thanks!

arijitde92 commented 7 months ago

Hi @TG1999 ,

After inspecting the cause of test failure, I found that a few test cases are failing but those test cases are not written by me. Please see screenshots below- macos13 ubuntu22 win2022

As you can see, the same tests are failing for all the OSes and none of them are written by me. Is this happening with other PRs too? Or is this happening somehow because of the code I wrote? As I am not able to find any relation between the code I wrote and the tests that are failing.

Please guide me on this. Thanks.

TG1999 commented 7 months ago

@arijitde92 please take a pull from latest main branch, main branch is green :)

arijitde92 commented 7 months ago

Hi @TG1999 , I have pulled the latest main branch in my forked repo as you can see the latest commit in the screenshot below. image

But still 1 test is failing in CI as shown in below screenshot. image

TG1999 commented 7 months ago

You have to run 'make valid' command to fix formatting errors

arijitde92 commented 7 months ago

Hi @TG1999 , checks are now passing. Let me know if anything else is required.

arijitde92 commented 6 months ago

Hi @TG1999 , I have added unit tests as requested and also edited the code as requested. But now some tests are failing again. Please check in the below screenshot- image