Skip to content

Conversation

@mostlyobvious
Copy link
Member

Reason:
rails/rails#31285

In short:

  • when in Rails, let Rails load the models after it loads ActiveRecord
    itself

  • when not Rails, proceed as usual

@mostlyobvious mostlyobvious force-pushed the lazy-load-active-record branch 2 times, most recently from 02e9961 to bc44078 Compare June 4, 2025 23:30
Reason:
rails/rails#31285

In short:

* when in Rails, let Rails load the models after it loads ActiveRecord
  itself

* when not Rails, proceed as usual
@mostlyobvious mostlyobvious force-pushed the lazy-load-active-record branch from 4ec000a to 07c8206 Compare June 5, 2025 06:34
@mostlyobvious mostlyobvious merged commit 1975827 into master Jun 5, 2025
55 checks passed
@mostlyobvious mostlyobvious deleted the lazy-load-active-record branch June 5, 2025 06:56
mostlyobvious added a commit that referenced this pull request Jul 9, 2025
Needs another take, this one was too simplistic.

      NameError: uninitialized constant RubyEventStore::ActiveRecord::WithDefaultModels::EventInStream (NameError)

            [Event, EventInStream]
                    ^^^^^^^^^^^^^

Related:
#1906
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