Skip to content

FDN-4581 Remove pr-validator workflow#132

Merged
flow-tech merged 1 commit intomainfrom
revert-pr-validator
Mar 6, 2026
Merged

FDN-4581 Remove pr-validator workflow#132
flow-tech merged 1 commit intomainfrom
revert-pr-validator

Conversation

@jackl
Copy link
Copy Markdown
Contributor

@jackl jackl commented Mar 6, 2026

Public repos cannot call reusable workflows from private repos (code-steward). Removing the pr-validator.yml that was added in error.

Summary by CodeRabbit

  • Chores
    • Removed GitHub Actions workflow for pull request validation.

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Mar 6, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 87edaf28-34ca-43d9-a74c-a583475ff97d

📥 Commits

Reviewing files that changed from the base of the PR and between 07e7039 and 25aa1b1.

📒 Files selected for processing (1)
  • .github/workflows/pr-validator.yml
💤 Files with no reviewable changes (1)
  • .github/workflows/pr-validator.yml

📝 Walkthrough

Walkthrough

Removed a GitHub Actions workflow file that previously delegated PR validation to an external reusable workflow. The deleted workflow monitored pull request events (opened, edited, synchronize) and executed validation checks through a centralized workflow reference.

Changes

Cohort / File(s) Summary
GitHub Actions Workflow
.github/workflows/pr-validator.yml
Deleted workflow file that delegated PR title and description validation to an external reusable workflow at flowcommerce/code-steward.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and specifically describes the main change: removing the pr-validator workflow, which aligns with the changeset that deletes the .github/workflows/pr-validator.yml file.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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


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

@flow-tech flow-tech merged commit 1dd7812 into main Mar 6, 2026
2 checks passed
@flow-tech flow-tech deleted the revert-pr-validator branch March 6, 2026 11:09
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.

2 participants