Skip to content

Drop Slim support - #113

Merged
AbuMareBear merged 1 commit into
mainfrom
remove-slim
Sep 24, 2026
Merged

AbuMareBear merged 1 commit into
mainfrom
remove-slim

Conversation

@AbuMareBear

Copy link
Copy Markdown
Contributor

Summary

Drops Slim support from a11y-lint. The gem now targets ERB and Phlex only.

Breaking change. .slim files are no longer discovered or linted. SlimRunner, SlimNode, and SlimLoadError are removed, along with the slim development dependency.

Removed

  • lib/a11y/lint/slim_runner.rb and lib/a11y/lint/slim_node.rb
  • All Slim test files (14), including the six unsuffixed perceivable rule tests that were Slim-only
  • The Slim dummy app fixture (home.html.slim)
  • slim from the gemspec and lockfile

Changed

  • CLI globs only .erb and .rb; .rb routes to PhlexRunner, everything else to ErbRunner. The "no files found" message now reads "No .erb or .rb files found"
  • CLI, configuration, and offense tests converted from Slim to ERB fixtures; the mixed-pipeline CLI test now covers ERB + Phlex
  • CHANGELOG entry under Unreleased
  • README, CLAUDE.md, .claude/rules/testing.md, and the implement/task commands no longer reference Slim; the testing rule's calling-style examples are now ERB
  • Docs site: Shared::CodeTabs drops its Slim tab and kwarg; rule pages lose their Slim samples and "Slim equivalent" sections; CONTRIBUTING updated

Test plan

  • bundle exec rake — 570 runs, 985 assertions, 0 failures; RuboCop clean
  • bundle exec a11y-lint test/fixtures/dummy_app — 38 offenses across ERB and Phlex fixtures, unchanged
  • A directory containing only a .slim file reports "No .erb or .rb files found"
  • Docs site build (cd docs && bin/bridgetown build) — not run locally; the component change is a one-tab removal

@AbuMareBear
AbuMareBear merged commit ac7ac22 into main Sep 24, 2026
1 check passed
@AbuMareBear
AbuMareBear deleted the remove-slim branch September 24, 2026 13:16
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