Skip to content

ci: reduce unnecessary workflow runs#42

Merged
trsdn merged 2 commits into
mainfrom
trsdn/save-github-runner-time
Jun 10, 2026
Merged

ci: reduce unnecessary workflow runs#42
trsdn merged 2 commits into
mainfrom
trsdn/save-github-runner-time

Conversation

@trsdn

@trsdn trsdn commented Jun 10, 2026

Copy link
Copy Markdown
Owner

Summary

  • Add PR path filters so heavyweight workflows only run when relevant files change
  • Remove duplicate dependency-review trigger
  • Disable scheduled test, security, docs, dashboard, and version-bump runs for this inactive repo

Validation

  • Parsed all workflow YAML files
  • Confirmed no remaining schedule: or cron: triggers
  • Ran git diff --check

Add path filters to heavyweight PR workflows, remove duplicate dependency review triggers, and disable scheduled maintenance runs for the inactive repo.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@github-actions

github-actions Bot commented Jun 10, 2026

Copy link
Copy Markdown
Contributor

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Scanned Files

None

Upgrade pip before pip-audit so the job does not fail on a vulnerable runner-provided pip version, and install the package documentation extra before pdoc imports the project.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@github-actions

Copy link
Copy Markdown
Contributor

🔍 PR Analysis Results

PR: #42 | Commit: a7e7c18bec1932303e0e869d05434e7d62e9a557

🎨 Code Formatting

All files properly formatted

🔧 Code Linting

No linting issues found

📝 Type Checking

Type checking issues found

Click to see type issues
usage: mypy [-h] [-v] [-V] [more options; see below]
            [-m MODULE] [-p PACKAGE] [-c PROGRAM_TEXT] [files ...]
mypy: error: unrecognized arguments: --json-report mypy_report.json

Fix: Add proper type annotations and resolve type errors

🔒 Security Analysis

No security issues detected

📊 Test Coverage Analysis

Coverage 80.48289738430583% meets 80% requirement

🧹 Dead Code Analysis

Dead code analysis completed

📋 Summary

⚠️ Found 1 issue(s) that should be addressed:

  • 📝 Types: Issues found

🔧 Quick Fix Commands:

# Fix formatting and auto-fixable linting issues
ruff format .
ruff check . --fix

# Run tests with coverage
pytest tests/unit/ --cov=markitdown_mcp --cov-report=term-missing

# Check security
bandit -r markitdown_mcp/

This analysis was automatically generated by the PR feedback workflow.
Report generated at 2026-06-10 09:23:10 UTC

@github-actions

Copy link
Copy Markdown
Contributor

🔍 CI Quality Gates Summary

Overall Status: ✅ All Passed

Check Status Details Action Required
🎨 Format ✅ Passed ruff format check None
🔧 Lint ✅ Passed ruff linting None
📝 Types ✅ Passed mypy type checking None
🧪 Tests ✅ Passed Unit tests None
📊 Coverage 80.5% Minimum: 80% None
🔌 MCP ✅ Valid Protocol compliance None
🔒 Security ✅ Clean Dependency audit None

🔗 Quick Links

🛠️ Quick Fix Commands

# Fix most issues automatically
ruff format .
ruff check . --fix

# Run tests locally
pytest tests/unit/ --cov=markitdown_mcp

# Check types
mypy markitdown_mcp

Last updated: 2026-06-10 09:24:44 UTC

@github-actions

Copy link
Copy Markdown
Contributor

🔍 PR Quality Summary

CI Status

✅ Security: success
✅ Docs: success
✅ Quality: success

Metrics

Metric Value Trend
📊 Coverage N/A -
🧪 Tests Test results unavailable -
⏱️ Performance No performance data -

Quality Checks

  • Format & Lint: Ruff formatting and linting
  • Type Safety: MyPy strict type checking
  • Security: Bandit, Safety, GitLeaks scanning
  • MCP Protocol: Tool schema validation
  • Documentation: Docstring coverage (80%+)

MCP Tools

  • convert_file - Convert individual files to Markdown
  • convert_directory - Batch convert directories
  • list_supported_formats - Query supported file types

🤖 Auto-generated by CI • Last updated: 2026-06-10 09:26 UTC

@trsdn trsdn merged commit 0c35234 into main Jun 10, 2026
63 checks passed
@trsdn trsdn deleted the trsdn/save-github-runner-time branch June 10, 2026 09:55
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.

1 participant