Skip to content

Don't use slim Ubuntu runner for pre-commit#376

Merged
5ouma merged 1 commit intomainfrom
ci-pre-commit-runner
Jan 31, 2026
Merged

Don't use slim Ubuntu runner for pre-commit#376
5ouma merged 1 commit intomainfrom
ci-pre-commit-runner

Conversation

@5ouma
Copy link
Owner

@5ouma 5ouma commented Jan 31, 2026

⚠️ Issue

close #


✏️ Description

It's quite slow to run.


Copilot AI review requested due to automatic review settings January 31, 2026 13:53
@coderabbitai
Copy link

coderabbitai bot commented Jan 31, 2026

Walkthrough

The pull request updates the GitHub Actions workflow configuration to use Ubuntu-Latest as the runner image for the pre-commit job, replacing the previous Ubuntu-Slim image. No changes to control flow or timeout settings.

Changes

Cohort / File(s) Summary
GitHub Actions Workflow Configuration
.github/workflows/pre-commit.yml
Updated runner image from Ubuntu-Slim to Ubuntu-Latest for the pre-commit job.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title directly describes the main change: switching from slim Ubuntu runner to standard Ubuntu-Latest for the pre-commit workflow job.
Description check ✅ Passed The description is related to the changeset, providing a rationale (slowness) for why the slim Ubuntu runner was replaced, though it lacks technical detail.
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.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch ci-pre-commit-runner

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

@github-actions github-actions bot added the 🎽 CI Changes to CI configuration files and scripts label Jan 31, 2026
Copy link

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

This PR updates the pre-commit workflow to use the Ubuntu-Latest runner instead of Ubuntu-Slim to improve workflow execution performance. The change addresses slow execution times experienced with the slim Ubuntu runner.

Changes:

  • Changed the GitHub Actions runner from Ubuntu-Slim to Ubuntu-Latest in the pre-commit workflow

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

@5ouma
Copy link
Owner Author

5ouma commented Jan 31, 2026

@coderabbitai review

@coderabbitai
Copy link

coderabbitai bot commented Jan 31, 2026

✅ Actions performed

Review triggered.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

@5ouma 5ouma merged commit 1daddbb into main Jan 31, 2026
13 checks passed
@5ouma 5ouma deleted the ci-pre-commit-runner branch January 31, 2026 14:19
@5ouma 5ouma bot mentioned this pull request Jan 31, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🎽 CI Changes to CI configuration files and scripts

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant