Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: Remove auto-assign CI job #3482

Merged
merged 1 commit into from
Dec 10, 2024
Merged

chore: Remove auto-assign CI job #3482

merged 1 commit into from
Dec 10, 2024

Conversation

m-horky
Copy link
Contributor

@m-horky m-horky commented Dec 9, 2024

The workflow has had lesser and lesser impact on the ways our team picks tasks waiting for review. We are beyond the point of diminishing returns, it's easier to drop it completely and rely on other means.

The workflow has had lesser and lesser impact on the ways our team picks tasks
waiting for review. We are beyond the point of diminishing returns, it's
easier to drop it completely and rely on other means.
Copy link

github-actions bot commented Dec 9, 2024

Coverage

Coverage (computed on Fedora latest) •
FileStmtsMissCoverMissing
TOTAL17674455574% 
report-only-changed-files is enabled. No files were changed during this commit :)

Tests Skipped Failures Errors Time
2470 14 💤 0 ❌ 0 🔥 30.997s ⏱️

Copy link
Contributor

@ptoscano ptoscano left a comment

Choose a reason for hiding this comment

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

The message nicely describes the situation, and thus I'm for it as well.

The github action even fails for contributors not in the client-tools group of the candlepin organization, which is even slightly annoying. Also candlepin itself moved away from it, so there is not much drive to fix this behaviour.

LGTM; waiting for Jiri's feedback.

Copy link
Contributor

@jirihnidek jirihnidek left a comment

Choose a reason for hiding this comment

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

LGTM

@jirihnidek jirihnidek merged commit 50c0d8c into main Dec 10, 2024
24 checks passed
@jirihnidek jirihnidek deleted the mhorky/no-auto-assign branch December 10, 2024 07:53
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.

3 participants