Skip to content

feat(ci): create shared pr-quality workflow#25

Merged
alexander-akait merged 2 commits intowebpack:mainfrom
avivkeller:main
Apr 12, 2026
Merged

feat(ci): create shared pr-quality workflow#25
alexander-akait merged 2 commits intowebpack:mainfrom
avivkeller:main

Conversation

@avivkeller
Copy link
Copy Markdown
Member

@avivkeller avivkeller commented Apr 3, 2026

cc @alexander-akait - Creates a shared workflow for pr-quality

Copilot AI review requested due to automatic review settings April 3, 2026 14:55
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Creates a reusable GitHub Actions workflow (workflow_call) intended to enforce PR quality checks (via peakoss/anti-slop) for pull requests in this repository.

Changes:

  • Adds a shared PR Quality workflow callable from other workflows.
  • Introduces an anti-slop job that checks PRs for template/conventional title/commit compliance and applies stricter thresholds for new contributors.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .github/workflows/pr-quality.yml
Copy link
Copy Markdown
Member

@alexander-akait alexander-akait left a comment

Choose a reason for hiding this comment

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

In webpack we have a little bit different police for contributor, I think ideally we should use the same across all repos

@avivkeller
Copy link
Copy Markdown
Member Author

I’m not following your review, is it blocking this, I don’t understand the concern?

@evenstensberg
Copy link
Copy Markdown
Member

@avivkeller what he means is that we have workflows standardized across the org and maybe some maintainers of certain repos don't like it

@avivkeller
Copy link
Copy Markdown
Member Author

I see. Unlike other .github files, workflows are not synced across the entire organization.

Repos still need to 'opt-in' to this workflow by calling it.

Copy link
Copy Markdown
Member

@alexander-akait alexander-akait left a comment

Choose a reason for hiding this comment

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

Let's use this in webpack and webpack docs repo

@alexander-akait alexander-akait merged commit a03552c into webpack:main Apr 12, 2026
1 check passed
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.

4 participants