rails_event_store
rails_event_store copied to clipboard
Rename RailsEventStoreActiveRecord into RubyEventStore::ActiveRecord
Creating an issue for something to track before 3.0. I'm not sure it makes sense to start working on the code before we decide we want to finally go into 3.x :)
https://github.com/RailsEventStore/rails_event_store/discussions/1147
Correct me if I'm wrong. As of now:
rails_event_store-2.6.0depends onrails_event_store-2.6.0rails_event_store-masterdepends onruby_event_store-active_record-master
We did it because we control this dependency and replaced internally all invocations of RailsEventStoreActiveRecord with RubyEventStore::ActiveRecord.
We're also providing rails_event_store_active_record gem that:
- has dependency to
ruby_event_store-active_record - aliases
RubyEventStore::ActiveRecordasRailsEventStoreActiveRecord - warns about rename in post-install message and when requiring
rails_event_store_active_record
The only reason this aliased gem exists is for someone directly depending on it (which we don't control).
There's also the case of documentation pointing to RailsEventStoreActiveRecord, which we don't want to change in 2.x. Also someone might have customized RailsEventStoreActiveRecord so this const being stil present is fair expectation.
Should we then keep rails_event_store to depend on rails_event_store_active_record, being a wrapper on ruby_event_store-active_record? Do we hide deprecation warnings? Do we need to mention this deprecation in the docs?
@swistak35 @lukaszreszke
Well, if we want to provide backwards compatibility, as far as I understand:
- we should still have a dependency on
rails_event_store_active_record - we should remove deprecation warnings
- with 3.0 release, we change the dependency to
ruby_event_store-active_record. It would be nice (but not mandatory) to also release 3.0 for bothruby_event_store-active_recordandrails_event_store_active_record(and 3.0 being the last version of this one). Then people not using rails_event_store could make gradual change (since they have to explicitly specifyrails_event_store_active_recordif they use it)
I don't see other reason since we can't say "either that gem or this gem" in gemspec, or "if we have ruby_event_store-active_record, then we rails_event_store_active_record is no longer dependency".
I don't see value in deprecation warnings in 2.x, even if it would be last version of 2.x. Unless people can actually solve the deprecation (by using newer version), but they can't (they can only do that when being on 3.0, without rails_event_store).