Skip to content

fix(ci): update pythinker-home dispatch target to TechMatrix-labs org - #8

Merged
elkaix merged 4 commits into
mainfrom
fix/dispatch-pythinker-home-org
May 28, 2026
Merged

fix(ci): update pythinker-home dispatch target to TechMatrix-labs org#8
elkaix merged 4 commits into
mainfrom
fix/dispatch-pythinker-home-org

Conversation

@elkaix

@elkaix elkaix commented May 28, 2026

Copy link
Copy Markdown
Member

Update the dispatch URL in dispatch-pythinker-home-sync.yml from mohamed-elkholy95/pythinker-home to TechMatrix-labs/pythinker-home following the repo transfer.

Summary by CodeRabbit

  • Documentation

    • Clarified dependency governance policies and approval requirements in contribution guidelines.
    • Updated repository guidance on dependency management and review standards.
  • Chores

    • Updated code review configuration for enhanced feedback quality and automated checks.
    • Updated GitHub workflow dispatch endpoint for project synchronization.

Review Change Stack

elkaix added 4 commits May 28, 2026 17:21
Schema-validated .coderabbit.yaml with assertive review profile,
Python-specific tool tuning (ruff on, flake8/pylint off), security
scanners (semgrep, trufflehog, gitleaks, trivy, zizmor), path
instructions for telemetry PII, type safety, CI workflow security,
and the zero-new-runtime-deps policy gate.
@coderabbitai

coderabbitai Bot commented May 28, 2026

Copy link
Copy Markdown
Contributor

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: d8cbedd8-1f0c-4cf1-9ecb-0e83544e89ca

📥 Commits

Reviewing files that changed from the base of the PR and between 898917d and e0bbd3a.

📒 Files selected for processing (4)
  • .coderabbit.yaml
  • .github/workflows/dispatch-pythinker-home-sync.yml
  • AGENTS.md
  • CONTRIBUTING.md

📝 Walkthrough

Walkthrough

Configuration and governance update establishing CodeRabbit code review rules with path-scoped security and dependency checks, documenting a zero-new-bundled-deps runtime dependency policy for contributors, and updating a GitHub Actions workflow dispatch endpoint to a new repository owner.

Changes

Code Review Setup & Dependency Governance

Layer / File(s) Summary
CodeRabbit Configuration with Path-Scoped Review Rules
.coderabbit.yaml
Full configuration sets en-US language and direct tone, enables review/auto-review/pre-merge features with collapsed walkthroughs and effort estimation, defines path-scoped instructions for telemetry, Python code, tests, GitHub workflows, and pyproject.toml with checks for PII/secrets, missing type annotations, unsafe function calls (eval, exec, subprocess), unlogged exceptions, unpinned workflow actions, and dependency justification; configures ruff, yamllint, markdownlint, actionlint, hadolint, shellcheck, and security scanners; enables chat auto-replies and pytest-based code generation guidance.
Dependency Governance Policy Documentation
AGENTS.md, CONTRIBUTING.md
AGENTS.md non-negotiable rules now explicitly reference zero-new-bundled-deps policy for [project].dependencies entries. CONTRIBUTING.md adds "Adding dependencies" section requiring maintainer approval via issue discussion, inline justification comment template, and notes that dev-only dependency groups are exempt from the policy.

Workflow Dispatch Endpoint Update

Layer / File(s) Summary
Repository Dispatch Endpoint Migration
.github/workflows/dispatch-pythinker-home-sync.yml
Updated repository dispatch target from mohamed-elkholy95/pythinker-home to TechMatrix-labs/pythinker-home.

🎯 3 (Moderate) | ⏱️ ~20 minutes

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Description check ❓ Inconclusive Description is minimal but explains the specific change. Missing required checklist items and issue link, though the change is straightforward and the core purpose is clear. Add the full checklist from the template and link the related issue (if any) to confirm all requirements are met.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed Title follows conventional commits format with valid type (fix), scope (ci), and clearly describes the main changeset: updating the dispatch target.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/dispatch-pythinker-home-org

Comment @coderabbitai help to get the list of available commands and usage tips.

@coderabbitai

coderabbitai Bot commented May 28, 2026

Copy link
Copy Markdown
Contributor

Actionable comments posted: 0

@elkaix
elkaix merged commit 9ccffbc into main May 28, 2026
21 checks passed
@elkaix
elkaix deleted the fix/dispatch-pythinker-home-org branch July 17, 2026 20:04
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