Skip to content

Follow up CI migration#351

Merged
5ouma merged 3 commits intomainfrom
chore-ci-update
Dec 6, 2025
Merged

Follow up CI migration#351
5ouma merged 3 commits intomainfrom
chore-ci-update

Conversation

@5ouma
Copy link
Owner

@5ouma 5ouma commented Dec 6, 2025

⚠️ Issue

close #


✏️ Description

  • Update timeout minutes
  • Update the README badge link

5ouma added 2 commits December 6, 2025 16:20
Some jobs has too much minutes to wait.
Point to CI not previous test.
Copilot AI review requested due to automatic review settings December 6, 2025 07:22
@coderabbitai
Copy link

coderabbitai bot commented Dec 6, 2025

Warning

Rate limit exceeded

@5ouma has exceeded the limit for the number of commits or files that can be reviewed per hour. Please wait 9 minutes and 54 seconds before requesting another review.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

📥 Commits

Reviewing files that changed from the base of the PR and between 6f6cd2f and ba20eeb.

📒 Files selected for processing (2)
  • .github/workflows/ci.yml (2 hunks)
  • docs/README.md (1 hunks)

Note

Other AI code review bot(s) detected

CodeRabbit has detected other AI code review bot(s) in this pull request and will avoid duplicating their findings in the review comments. This may lead to a less comprehensive review.

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

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 🎽 CI Changes to CI configuration files and scripts 📖 Documentation Documentation only changes labels Dec 6, 2025
@codecov
Copy link

codecov bot commented Dec 6, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 20.44%. Comparing base (6f6cd2f) to head (ba20eeb).
⚠️ Report is 3 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #351   +/-   ##
=======================================
  Coverage   20.44%   20.44%           
=======================================
  Files           9        9           
  Lines         181      181           
=======================================
  Hits           37       37           
  Misses        130      130           
  Partials       14       14           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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 completes the CI migration by updating references from the old test.yml workflow to the new ci.yml workflow and optimizing timeout values based on actual job performance.

  • Updates the README badge to reference the new ci.yml workflow instead of test.yml
  • Reduces timeout values for the test job (3→1 minutes) and dry-run-publish job (3→2 minutes) to better reflect actual execution times

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
docs/README.md Updates CI workflow badge from test.yml to ci.yml
.github/workflows/ci.yml Optimizes timeout minutes for test and dry-run-publish jobs

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

Make the file name lowercase.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Signed-off-by: Souma <101255979+5ouma@users.noreply.github.com>
@5ouma 5ouma merged commit e2d9296 into main Dec 6, 2025
14 of 15 checks passed
@5ouma 5ouma deleted the chore-ci-update branch December 6, 2025 07:29
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 📖 Documentation Documentation only changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants