Skip to content

Replace migration template with standard Rails engine migration - #37

Merged
tylercschneider merged 1 commit into
mainfrom
feature/standard-migrations
Mar 19, 2026
Merged

tylercschneider merged 1 commit into
mainfrom
feature/standard-migrations

Conversation

@tylercschneider

Copy link
Copy Markdown
Collaborator

Summary

  • Moved from a generator-copied .tt migration template to a real migration file in db/migrate/
  • Host apps now use the standard rails herald:install:migrations rake task to copy migrations
  • Simplified the install generator by removing copy_migration and the ActiveRecord::Generators::Migration dependency
  • Updated README install instructions and generator tests

Test plan

  • Full test suite passes (200 tests, 0 failures)
  • Verify rails herald:install:migrations copies the migration in a host app
  • Verify rails generate herald:install still creates initializer and mounts routes

🤖 Generated with Claude Code

Move from generator-copied .tt template to a real migration in db/migrate/,
enabling the standard `rails herald:install:migrations` workflow.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@tylercschneider
tylercschneider merged commit d509e5b into main Mar 19, 2026
1 check passed
@tylercschneider
tylercschneider deleted the feature/standard-migrations branch March 19, 2026 16:52
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