Merge pull request #255 from thomasscheer/fixIssue240

Remove None wheel URLs before sorting them

Author: TG1999

.github/workflows/docs-ci.yml

@@ -13,10 +13,10 @@ jobs:
 
     steps:
       - name: Checkout code
-        uses: actions/checkout@v4
+        uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd  # v6.0.2
 
       - name: Set up Python ${{ matrix.python-version }}
-        uses: actions/setup-python@v5
+        uses: actions/setup-python@a309ff8b426b58ec0e2a45f0f869d46889d02405 # v6.2.0
         with:
           python-version: ${{ matrix.python-version }}
 

.github/workflows/pypi-release.yml

@@ -24,9 +24,9 @@ jobs:
     runs-on: ubuntu-24.04
 
     steps:
-      - uses: actions/checkout@v4
+      - uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd  # v6.0.2
       - name: Set up Python
-        uses: actions/setup-python@v5
+        uses: actions/setup-python@a309ff8b426b58ec0e2a45f0f869d46889d02405 # v6.2.0
         with:
           python-version: 3.14
 

src/python_inspector/package_data.py

@@ -122,6 +122,8 @@ async def get_pypi_data_from_purl(
 
 
 def choose_single_wheel(wheel_urls: List[str]) -> Optional[str]:
+    # fix Issue 240 - TypeError: '<' not supported between instances of 'str' and 'NoneType'
+    wheel_urls = [url for url in wheel_urls if url is not None]
     """
     Sort wheel urls descendingly and return the first one
     """

tests/data/azure-devops.req-310-expected.json

@@ -2,7 +2,6 @@
   "headers": {
     "tool_name": "python-inspector",
     "tool_homepageurl": "https://github.com/aboutcode-org/python-inspector",
-    "tool_version": "0.13.0",
     "options": [
       "--index-url https://pypi.org/simple",
       "--json <file>",

tests/data/azure-devops.req-312-expected.json

@@ -2,7 +2,6 @@
   "headers": {
     "tool_name": "python-inspector",
     "tool_homepageurl": "https://github.com/aboutcode-org/python-inspector",
-    "tool_version": "0.13.0",
     "options": [
       "--index-url https://pypi.org/simple",
       "--json-pdt <file>",