Skip to content

fix: update croc to v11.5.2 - #9

Open
schollz wants to merge 1 commit into
interfluve-wav:mainfrom
schollz:schollz/update-croc
Open

schollz wants to merge 1 commit into
interfluve-wav:mainfrom
schollz:schollz/update-croc

Conversation

@schollz

@schollz schollz commented Sep 13, 2026 •

Copy link
Copy Markdown

Update bundled croc to v11.5.2 for recent security updates; it would be good to ship this soon. Updates both workflows and the download script.

Validated: download, script syntax, and bundled binary version (11.5.2).

@coderabbitai

coderabbitai Bot commented Sep 13, 2026 •

Copy link
Copy Markdown

Review Change StackReview 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: 1b546484-f613-45c8-8ab0-681536e867a0

📥 Commits

Reviewing files that changed from the base of the PR and between f0cef56 and 5e33c7a.

📒 Files selected for processing (3)
  • .github/workflows/build-gui.yml
  • .github/workflows/release-gui.yml
  • gui/scripts/bundle-croc.mjs

Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review.


📝 Walkthrough

Walkthrough

The GUI build and release workflows now select croc v11.5.2. The bundling script uses v11.5.2 when CROC_VERSION is unset or empty.

Changes

Croc version alignment

Layer / File(s) Summary
Update croc release version
.github/workflows/build-gui.yml, .github/workflows/release-gui.yml, gui/scripts/bundle-croc.mjs
The workflows and bundling fallback now use croc v11.5.2 instead of v11.0.0.

Priority: ➖ Normal

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

Change: Other

Merge Risk: ⚪ Minimal · up to 5e33c

The croc version is consistently updated across the workflows and bundling fallback, with the reported download and version checks passing.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 1 functions across 1 files. (2 skipped: 2 … Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (4 passed)
Check name Status Explanation
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.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and concisely describes the main change: updating the bundled croc version to v11.5.2.
Full details: Docstring Coverage

Explanation

Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 1 functions across 1 files. (2 skipped: 2 unsupported.)

  • Fix all pre-merge checks with AI
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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.

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