Skip to content

Enable GitHub Actions CI#12

Merged
mattwebbio merged 1 commit into
mainfrom
fix-bedrock-converse-multi-block-thinking-replay
Jul 9, 2026
Merged

Enable GitHub Actions CI#12
mattwebbio merged 1 commit into
mainfrom
fix-bedrock-converse-multi-block-thinking-replay

Conversation

@mattwebbio

@mattwebbio mattwebbio commented Jul 9, 2026

Copy link
Copy Markdown

Summary

Actions had been disabled org-wide on this repo, so the CI workflows already checked into .github/workflows/ had never actually run. Now that Actions is enabled:

  • Dropped the Codecov upload step and RUBYGEMS_AUTH_TOKEN publish step from _test-matrix.yml/release.yml — this fork isn't configuring either secret. The GitHub Packages publish step stays; it authenticates with the built-in GITHUB_TOKEN.
  • Removed .github/workflows/docs.yml — it's entirely GitHub Pages-dependent (the build job's configure-pages step 404s without Pages enabled), and Pages isn't being enabled either.
  • Pinned rdoc below 8 in the Gemfile. rdoc 8.0 pulls in rbs (transitively, via irbrailties), whose native C extension doesn't build on JRuby — this broke bundle install on that leg of the test matrix the first time CI actually ran. Dev-only pin, no runtime impact.

Test plan

  • bundle exec rubocop — clean, repo-wide
  • bundle exec rspec — full suite green (1633 examples, 0 failures)
  • CI green end-to-end on this branch: gitleaks, RuboCop, and the full Ruby/Rails test matrix including JRuby

🤖 Generated with Claude Code

@mattwebbio mattwebbio requested a review from a team as a code owner July 9, 2026 20:36
@staging-supernova-dx-appf-io

Copy link
Copy Markdown

🤖 Supernova Code Review — View trace

@mattwebbio mattwebbio force-pushed the fix-bedrock-converse-multi-block-thinking-replay branch 3 times, most recently from b110b19 to 7725240 Compare July 9, 2026 22:01
Drop the Codecov and RubyGems.org publish steps since this fork won't
configure those secrets, and remove the Pages-only docs workflow since
GitHub Pages isn't being enabled either.

Also pin rdoc below 8, since rdoc 8.0 pulls in rbs (a transitive dep
of irb, via railties) whose native extension doesn't build on JRuby,
breaking bundle install on that leg of the test matrix. This was
already latent since Actions had never actually run before now.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@mattwebbio mattwebbio force-pushed the fix-bedrock-converse-multi-block-thinking-replay branch from 7725240 to f70cdd1 Compare July 9, 2026 22:13
@mattwebbio mattwebbio changed the title Fix Bedrock Converse streaming losing reasoning blocks on replay Enable GitHub Actions CI Jul 9, 2026
@mattwebbio mattwebbio merged commit 2161bfe into main Jul 9, 2026
21 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