Skip to content

feat(ci): migrate from CircleCI to GitHub Actions#20

Merged
sabjorn merged 4 commits intomainfrom
LUM-195-github-actions-migration-follow-up-work
Mar 11, 2026
Merged

feat(ci): migrate from CircleCI to GitHub Actions#20
sabjorn merged 4 commits intomainfrom
LUM-195-github-actions-migration-follow-up-work

Conversation

@sabjorn
Copy link

@sabjorn sabjorn commented Mar 10, 2026

Summary

  • Add ci.yml workflow with lint and test jobs running on PRs to main
  • Bump actions/checkout and actions/setup-node from v3 to v4 in build/release workflows
  • Add yarn caching to build workflow
  • Remove .circleci/ directory (config + Docker images)

Test plan

  • Verify CI workflow triggers on PR and runs lint + test jobs
  • Verify build workflow still produces prebuilds correctly
  • Verify release workflow still publishes on main

🤖 Generated with Claude Code

Marvin Martian and others added 4 commits March 10, 2026 12:05
Add ci.yml with lint and test jobs for PRs. Bump actions to v4 and add
yarn caching to build/release workflows.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
CI has been migrated to GitHub Actions.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Static FFmpeg builds from Lumen5/ffmpeg-static do not include
OpenSSL/GnuTLS, so HTTPS protocol is unavailable. Skip affected
tests when FFMPEG_STATIC=1 is set.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Static FFmpeg doesn't include libx264, so the default h264 encoder
varies by platform (e.g. h264_v4l2m2m on Linux). Relax the assertion
when FFMPEG_STATIC=1.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@sabjorn sabjorn marked this pull request as ready for review March 10, 2026 23:17
const beamcoder = require('../index.js');

test('Creating a demuxer', async t => {
// Known failure with static FFmpeg builds: HTTPS protocol requires OpenSSL/GnuTLS

Choose a reason for hiding this comment

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

we need to fix it in order to run tests with static linking

@sabjorn sabjorn merged commit 4c69de1 into main Mar 11, 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.

2 participants