Skip to content

Use gh-ci-action v1.1.0 - #76

Merged
davesnx merged 1 commit into
mainfrom
ci/gh-ci-action-v1.1.0
Oct 21, 2026
Merged

davesnx merged 1 commit into
mainfrom
ci/gh-ci-action-v1.1.0

Conversation

@davesnx

@davesnx davesnx commented Oct 21, 2026 •

Copy link
Copy Markdown
Owner

Use gh-ci-action v1.1.0:

  • A pin on a branch or tag now gets a new cache key when the branch moves.
  • On Windows, the dependency install runs up to 3 times.
  • extra-packages are installed themselves, without the --with-* flags (v1.0.1).

The dune cache is now saved only when the job passes. After a failed cram test, the cache held the wrong output, and dune replayed it in the next run without running the test again (server-reason-react #407).

Risk: two-way door; a revert restores v1.0.0 and the old save condition.

Summary by CodeRabbit

  • Chores
    • Updated continuous integration tooling and build-cache handling to use newer action versions.
    • Adjusted cache-saving behavior to improve consistency during workflow runs.

@coderabbitai

coderabbitai Bot commented Oct 21, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration
  • Configuration used: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 610fd926-4b34-4a5e-a57f-a8828958bac6
📥 Commits

Reviewing files that changed from the base of the PR and between c6e56df and 470786a.

📒 Files selected for processing: 1
  • .github/workflows/ci.yml

Included review availability: This review used your included allowance. Your plan provides up to 4 included reviews per hour; 3 remain after this review.

📝 Walkthrough

The CI workflow updates the OCaml setup and dune-cache actions from v1.0.0 to v1.1.0. The dune-cache save step no longer uses the !cancelled() condition.

Changes: CI Workflow

Layer / File(s) Summary
Update CI actions
.github/workflows/ci.yml
The workflow uses v1.1.0 for OCaml setup and dune-cache actions. The dune-cache save step no longer declares the !cancelled() condition.

Priority: ⬇️ Low

Estimated code review effort: 1 (Trivial) | ~5 minutes

Change: Bug fix

Merge Risk: ⚪ Minimal · up to 47078

The CI action upgrade and cache-condition change are mergeable with no identified production or workflow risk.

Pre-merge checks: Passed 5
Check name Status Explanation
Description Check Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check Passed The title clearly and concisely identifies the main change: updating the project to use gh-ci-action v1.1.0.
Docstring Coverage Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check Passed Check skipped because no linked issues were found for this pull request.
Full details: Docstring Coverage

Explanation

No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (1 skipped: 1 unsupported.)

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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.

@davesnx
davesnx merged commit 5c0318b into main Oct 21, 2026
5 checks 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.

1 participant