Skip to content

fix(ci): replace disallowed julia-buildpkg/runtest/processcoverage actions and sync actions.lock - #70

Merged
hyperpolymath merged 1 commit into
mainfrom
fix/ci-allowed-julia-steps
Oct 24, 2026
Merged

hyperpolymath merged 1 commit into
mainfrom
fix/ci-allowed-julia-steps

Conversation

@hyperpolymath

Copy link
Copy Markdown
Owner

Replace org-disallowed julia-buildpkg/julia-runtest/julia-processcoverage steps with inline General + JPR + Pkg.test(coverage=true) and sync actions.lock.

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 24, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-24T11:16:46.318086Z 18e6b4b PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@coderabbitai

coderabbitai Bot commented Oct 24, 2026 •

Copy link
Copy Markdown
Contributor

Review in Change Stack →

Caution

Review failed

The pull request is closed.

📝 Summary

Summary by CodeRabbit

  • Chores
    • Continuous integration checks can now run for up to 30 minutes. Build, test and coverage checks run together after the required package registries are added and dependencies are installed. These changes affect automated project validation rather than the app’s user-facing features.

The CI workflow increases the test job timeout from 15 to 30 minutes. It replaces separate Julia build, test and coverage actions with a shell step that adds registries and runs package setup, build and tests with coverage enabled.

Changes: CI workflow

Layer / File(s) Summary
Julia CI execution
.github/workflows/CI.yml
The workflow adds the General and hyperpolymath registries, then runs Pkg.instantiate(), Pkg.build() and Pkg.test(coverage=true). The job timeout increases to 30 minutes.

Priority: Not assessed

Estimated code review effort: 2 (Simple) | ~10 minutes

Change: Bug fix

Possibly related issues

  • hyperpolymath/reposystem#319: The workflow replaces separate Julia actions with inline shell commands, which overlaps with the issue’s concern about disallowed workflow action usage. The provided changes do not show an update to actions.lock or address its data.
Merge Risk: Low · up to 18e6b

CI still tests the project, but it no longer produces the previous LCOV coverage report; restore processing if that report is required.

- [ ] 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.

@hyperpolymath
hyperpolymath merged commit 98f0ac0 into main Oct 24, 2026
15 of 27 checks passed
@hyperpolymath
hyperpolymath deleted the fix/ci-allowed-julia-steps branch October 24, 2026 11:18
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