Skip to content

Add Audition CI checks - #2286

Closed
joelhawksley wants to merge 3 commits into
marcoroth:mainfrom
joelhawksley:joelhawksley-add-audition-ci
Closed

Add Audition CI checks#2286
joelhawksley wants to merge 3 commits into
marcoroth:mainfrom
joelhawksley:joelhawksley-add-audition-ci

Conversation

@joelhawksley

@joelhawksley joelhawksley commented Aug 18, 2026

Copy link
Copy Markdown
Contributor

TL;DR

This PR adds Audition, a linter for enforcing Ractor compatilibilty. It also fixes a few low-hanging fruit.

Summary

  • add Audition 0.2.2 to the development bundle and CI
  • baseline 44 existing static findings so CI blocks new regressions
  • fix nine Ractor-readiness findings with shareable constants and direct process status handling
  • retain intentional lazy requires with documented inline suppressions

Testing

  • bundle exec rubocop lib/herb/bootstrap.rb lib/herb/cli.rb lib/herb/engine/error_formatter.rb lib/herb/engine/validation_error_overlay.rb lib/herb/project.rb
  • bundle exec rake test:engine test:integration
  • bundle exec audition . --static-only --format github

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 7d482578-5711-4d80-bb76-acd088a3a746
@github-actions github-actions Bot added ruby Ruby source for the gem and its libraries engine Herb engine and Rails template compilation rubygem The herb RubyGem and its packaging labels Aug 18, 2026
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 7d482578-5711-4d80-bb76-acd088a3a746
Keep Audition's Rubydex RBI files out of Sorbet's bundled dependency scan.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 7d482578-5711-4d80-bb76-acd088a3a746
@joelhawksley
joelhawksley marked this pull request as ready for review August 18, 2026 20:17
@marcoroth
marcoroth marked this pull request as draft August 21, 2026 03:31
@joelhawksley

Copy link
Copy Markdown
Contributor Author

Closing as it looks like #2263 and #2346 are the material improvements to compile time 鉂わ笍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

engine Herb engine and Rails template compilation ruby Ruby source for the gem and its libraries rubygem The herb RubyGem and its packaging

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant