Skip to content

use gha instead of circleci - #54

Merged
joshuafernandes merged 5 commits into
masterfrom
switch-to-gha
Jul 30, 2026
Merged

use gha instead of circleci#54
joshuafernandes merged 5 commits into
masterfrom
switch-to-gha

Conversation

@joshuafernandes

@joshuafernandes joshuafernandes commented Jul 28, 2026

Copy link
Copy Markdown
Contributor

Note

Medium Risk
Changes release automation and Cloudsmith publishing gates; mistakes could break builds or affect artifact publication, but no application/runtime crypto code is modified in this diff.

Overview
Adds a new GitHub Actions workflow at .github/workflows/ci.yml to replace CircleCI for building and publishing the jblst Java artifact with bundled native BLST libraries.

The pipeline runs on push/PR to master and semver tags, with parallel jobs for Linux x86_64, cross-compiled Linux ARM64, macOS (x86_64 + arm64 with arch directory fixes), and Windows (TDM GCC + Swig). Each platform job uploads native artifacts; an assemble job downloads them, copies generated Java/resources into src/main, runs ./gradlew build and checkJarContents, and stores the JAR plus workspace artifacts.

A gated publish job (protected publish environment, master / release-* / tags only) reuses the assembled workspace and runs checkJarContents publish to Cloudsmith using CLOUDSMITH_USER / CLOUDSMITH_API_KEY secrets.

Reviewed by Cursor Bugbot for commit e37e413. Bugbot is set up for automated code reviews on this repo. Configure here.

Comment thread .github/workflows/ci.yml Fixed
Comment thread .github/workflows/ci.yml Fixed
Comment thread .github/workflows/ci.yml Fixed
Comment thread .github/workflows/ci.yml Fixed
Comment thread .github/workflows/ci.yml Fixed
Comment thread .github/workflows/ci.yml Fixed
Comment thread .github/workflows/ci.yml Outdated
Comment thread .github/workflows/ci.yml
on:
push:
branches:
- master

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Release branches never trigger CI

Medium Severity

The workflow on.push.branches list only includes master, but the publish job if still allows refs/heads/release-*. Pushes to release branches never start the workflow, so that publish path from the old CircleCI filters is unreachable.

Additional Locations (1)
Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit 77a7bc3. Configure here.

Comment thread .github/workflows/ci.yml
Comment thread .github/workflows/ci.yml Outdated
Comment thread .github/workflows/ci.yml Outdated
Comment thread .github/workflows/ci.yml
Comment thread .github/workflows/ci.yml Outdated
Comment thread .github/workflows/ci.yml

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

There are 3 total unresolved issues (including 2 from previous reviews).

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 1256aeb. Configure here.

Comment thread .github/workflows/ci.yml
@joshuafernandes
joshuafernandes force-pushed the switch-to-gha branch 2 times, most recently from 7726d4d to 1e12bd1 Compare July 30, 2026 00:26
@joshuafernandes
joshuafernandes merged commit 4bb192d into master Jul 30, 2026
9 checks passed
@github-actions github-actions Bot locked and limited conversation to collaborators Jul 30, 2026
@joshuafernandes
joshuafernandes deleted the switch-to-gha branch July 30, 2026 01:16
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants