ultralytics-actions 0.0.36__tar.gz → 0.0.38__tar.gz

{ultralytics_actions-0.0.36 → ultralytics_actions-0.0.38}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: ultralytics-actions
-Version: 0.0.36
+Version: 0.0.38
 Summary: Ultralytics Actions for GitHub automation and PR management.
 Author-email: Glenn Jocher <glenn.jocher@ultralytics.com>
 Maintainer-email: Ultralytics <hello@ultralytics.com>
@@ -99,7 +99,7 @@ To use this action in your Ultralytics repository:
              labels: true # autolabel issues and PRs
              python: true # format Python code and docstrings
              prettier: true # format YAML, JSON, Markdown and CSS
-             swift: true # format Swift code (requires 'macos-latest')
+             swift: true # format Swift code (requires 'macos-latest' runner)
              spelling: true # check spelling
              links: true # check broken links
              summary: true # print PR summary with GPT4o (requires 'openai_api_key')

{ultralytics_actions-0.0.36 → ultralytics_actions-0.0.38}/README.md

@@ -63,7 +63,7 @@ To use this action in your Ultralytics repository:
              labels: true # autolabel issues and PRs
              python: true # format Python code and docstrings
              prettier: true # format YAML, JSON, Markdown and CSS
-             swift: true # format Swift code (requires 'macos-latest')
+             swift: true # format Swift code (requires 'macos-latest' runner)
              spelling: true # check spelling
              links: true # check broken links
              summary: true # print PR summary with GPT4o (requires 'openai_api_key')

{ultralytics_actions-0.0.36 → ultralytics_actions-0.0.38}/actions/__init__.py

@@ -22,4 +22,4 @@
 #     ├── test_summarize_pr.py
 #     └── ...
 
-__version__ = "0.0.36"
+__version__ = "0.0.38"

{ultralytics_actions-0.0.36 → ultralytics_actions-0.0.38}/actions/utils/common_utils.py

@@ -50,7 +50,8 @@ def is_url(url, check=True, max_attempts=3, timeout=2):
 
         # Check structure
         result = parse.urlparse(url)
-        if not all([result.scheme, result.netloc]):
+        partition = result.netloc.partition(".")  # i.e. netloc = "github.com" -> ("github", ".", "com")
+        if not result.scheme or not partition[0] or not partition[2]:
             return False
 
         # Check response
@@ -58,10 +59,10 @@ def is_url(url, check=True, max_attempts=3, timeout=2):
             for attempt in range(max_attempts):
                 try:
                     headers = {
-                        "User-Agent": "Mozilla/5.0 (Windows NT 10.0; Win64; x64) Chrome/120.0.0.0",
-                        "Accept": "*",
-                        "Accept-Language": "*",
-                        "Accept-Encoding": "*",
+                        "User-Agent": "Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/120.0.0.0 Safari/537.36",
+                        "Accept": "*/*",  # Wildcard for maximum compatibility
+                        "Accept-Language": "*",  # Wildcard for any language
+                        "Accept-Encoding": "*",  # Wildcard for any encoding
                     }
                     return requests.head(url, headers=headers, timeout=timeout, allow_redirects=True).status_code < 400
                 except Exception:
@@ -82,7 +83,7 @@ def check_links_in_string(text, verbose=True, return_bad=False):
         r"("  # Start capturing group for plaintext URLs
         r"(?:https?://)?"  # Optional http:// or https://
         r"(?:www\.)?"  # Optional www.
-        r"[\w.-]+"  # Domain name and subdomains
+        r"(?:[\w.-]+)?"  # Optional domain name and subdomains
         r"\.[a-zA-Z]{2,}"  # TLD
         r"(?:/[^\s\"')\]]*)?"  # Optional path
         r")"
@@ -104,3 +105,7 @@ def check_links_in_string(text, verbose=True, return_bad=False):
         print(f"WARNING ⚠️ errors found in URLs {bad_urls}")
 
     return (passing, bad_urls) if return_bad else passing
+
+
+if __name__ == "__main__":
+    print(is_url("https://ultralytics.com/images/bus.jpg"))

{ultralytics_actions-0.0.36 → ultralytics_actions-0.0.38}/pyproject.toml

@@ -95,6 +95,10 @@ packages = { find = { where = ["."], include = ["actions", "actions.*"] } }
 [tool.setuptools.dynamic]
 version = { attr = "actions.__version__" }
 
+[tool.pytest.ini_options]
+addopts = "--doctest-modules --durations=30 --color=yes"
+norecursedirs = [".git", "dist", "build"]
+
 [tool.ruff]
 line-length = 120
 

ultralytics_actions-0.0.38/tests/test_urls.py

@@ -0,0 +1,99 @@
+# Ultralytics Actions 🚀, AGPL-3.0 license
+# Continuous Integration (CI) GitHub Actions tests
+
+import pytest
+
+from actions.utils.common_utils import check_links_in_string, is_url
+
+URLS = [
+    "https://docs.ultralytics.com/help/CLA/",
+    "https://docs.ultralytics.com/help/contributing",
+    "https://docs.ultralytics.com",
+    "https://ultralytics.com",
+    "https://ultralytics.com/images/bus.jpg",
+    "https://github.com/ultralytics/ultralytics",
+]
+
+
+@pytest.fixture
+def verbose():
+    """Fixture that provides a verbose logging utility for detailed output during testing and debugging."""
+    return False  # Set False to suppress print statements during tests
+
+
+def test_is_url():
+    """Test each URL using is_url function."""
+    for url in URLS:
+        assert is_url(url), f"URL check failed: {url}"
+
+
+def test_html_links(verbose):
+    """Tests the validity of URLs within HTML anchor tags and returns any invalid URLs found."""
+    text = "Visit <a href='https://err.com'>our site</a> or <a href=\"http://test.org\">test site</a>"
+    result, urls = check_links_in_string(text, verbose, return_bad=True)
+    assert result is False
+    assert set(urls) == {"https://err.com", "http://test.org"}
+
+
+def test_markdown_links(verbose):
+    """Validates URLs in markdown links within a given text using check_links_in_string."""
+    text = "Check [Example](https://err.com) or [Test](http://test.org)"
+    result, urls = check_links_in_string(text, verbose, return_bad=True)
+    assert result is False
+    assert set(urls) == {"https://err.com", "http://test.org"}
+
+
+def test_mixed_formats(verbose):
+    """Tests URL detection in mixed text formats (HTML, Markdown, plain text) using check_links_in_string."""
+    text = "A <a href='https://1.com'>link</a> and [markdown](https://2.org) and https://3.net"
+    result, urls = check_links_in_string(text, return_bad=True)
+    assert result is False
+    assert set(urls) == {"https://1.com", "https://2.org", "https://3.net"}
+
+
+def test_duplicate_urls(verbose):
+    """Tests detection of duplicate URLs in various text formats using the check_links_in_string function."""
+    text = "Same URL: https://err.com and <a href='https://err.com'>link</a>"
+    result, urls = check_links_in_string(text, verbose, return_bad=True)
+    assert result is False
+    assert set(urls) == {"https://err.com"}
+
+
+def test_no_urls(verbose):
+    """Tests that a string with no URLs returns True when checked using the check_links_in_string function."""
+    text = "This text contains no URLs."
+    result, urls = check_links_in_string(text, verbose, return_bad=True)
+    assert result is True
+    assert not set(urls)
+
+
+def test_invalid_urls(verbose):
+    """Test invalid URLs."""
+    text = "Invalid URL: http://.com"
+    result, urls = check_links_in_string(text, verbose, return_bad=True)
+    assert result is False
+    assert set(urls) == {"http://.com"}
+
+
+def test_urls_with_paths_and_queries(verbose):
+    """Test URLs with paths and query parameters to ensure they are correctly identified and validated."""
+    text = "Complex URL: https://err.com/path?query=value#fragment"
+    result, urls = check_links_in_string(text, verbose, return_bad=True)
+    assert result is False
+    assert set(urls) == {"https://err.com/path?query=value#fragment"}
+
+
+def test_urls_with_different_tlds(verbose):
+    """Test URLs with various top-level domains (TLDs) to ensure correct identification and handling."""
+    text = "Different TLDs: https://err.ml https://err.org https://err.net https://err.io https://err.ai"
+    result, urls = check_links_in_string(text, verbose, return_bad=True)
+    assert result is False
+    assert set(urls) == {"https://err.ml", "https://err.org", "https://err.net", "https://err.io", "https://err.ai"}
+
+
+def test_case_sensitivity(verbose):
+    """Tests URL case sensitivity by verifying that URLs with different cases are correctly identified and handled."""
+    text = "Case test: HTTPS://err.com and https://err.com"
+    result, urls = check_links_in_string(text, verbose, return_bad=True)
+    assert result is False
+    assert set(urls) == {"https://err.com"}

{ultralytics_actions-0.0.36 → ultralytics_actions-0.0.38}/ultralytics_actions.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: ultralytics-actions
-Version: 0.0.36
+Version: 0.0.38
 Summary: Ultralytics Actions for GitHub automation and PR management.
 Author-email: Glenn Jocher <glenn.jocher@ultralytics.com>
 Maintainer-email: Ultralytics <hello@ultralytics.com>
@@ -99,7 +99,7 @@ To use this action in your Ultralytics repository:
              labels: true # autolabel issues and PRs
              python: true # format Python code and docstrings
              prettier: true # format YAML, JSON, Markdown and CSS
-             swift: true # format Swift code (requires 'macos-latest')
+             swift: true # format Swift code (requires 'macos-latest' runner)
              spelling: true # check spelling
              links: true # check broken links
              summary: true # print PR summary with GPT4o (requires 'openai_api_key')

ultralytics_actions-0.0.36/tests/test_urls.py

@@ -1,15 +0,0 @@
-from actions.utils.common_utils import is_url
-
-URLS = [
-    "https://docs.ultralytics.com/help/CLA/",
-    "https://docs.ultralytics.com/help/contributing",
-    "https://docs.ultralytics.com",
-    "https://ultralytics.com",
-    "https://github.com/ultralytics/ultralytics",
-]
-
-
-def test_urls():
-    """Test each URL using is_url function."""
-    for url in URLS:
-        assert is_url(url), f"URL check failed: {url}"