Skip to content

ci: disable action doc link checks#2756

Open
topher-lo wants to merge 1 commit into
mainfrom
codex/disable-action-doc-link-tests
Open

ci: disable action doc link checks#2756
topher-lo wants to merge 1 commit into
mainfrom
codex/disable-action-doc-link-tests

Conversation

@topher-lo

@topher-lo topher-lo commented May 22, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Remove the dedicated test-action-doc-links job from the Python CI workflow.
  • Add a module-level skip to scripts/tests/test_action_doc_links.py so the action documentation link checks are disabled when invoked directly.

Validation

  • uv run pytest scripts/tests/test_action_doc_links.py -q reports 3 skipped tests.
  • git diff --check passes.
  • Commit hooks passed during git commit.

Summary by cubic

Disable action documentation link checks in CI to reduce build time and avoid flaky failures. Removes the test-action-doc-links job and adds a module-level skip so these tests are always skipped.

  • Removed the test-action-doc-links job from .github/workflows/test-python.yml.
  • Added pytestmark = pytest.mark.skip(...) in scripts/tests/test_action_doc_links.py.

Written for commit 19702f0. Summary will update on new commits. Review in cubic

@topher-lo topher-lo added cicd Changes to CICD workflows and configurations tests Changes to unit and integration tests labels May 22, 2026 — with ChatGPT Codex Connector
@zeropath-ai

zeropath-ai Bot commented May 22, 2026

Copy link
Copy Markdown

No security or compliance issues detected. Reviewed everything up to 19702f0.

Security Overview
Detected Code Changes
Change Type Relevant files
Configuration changes ► .github/workflows/test-python.yml
    Remove test-action-doc-links job
Refactor ► scripts/tests/test_action_doc_links.py
    Skip test-action-doc-links test

@topher-lo topher-lo marked this pull request as ready for review May 22, 2026 18:01
@topher-lo topher-lo requested a review from daryllimyt May 22, 2026 18:02

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

No issues found across 2 files

Confidence score: 5/5

  • Automated review surfaced no issues in the provided summaries.
  • No files require special attention.

Re-trigger cubic

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cicd Changes to CICD workflows and configurations tests Changes to unit and integration tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants