This project is archived and is in readonly mode.
Make timestamped migrations optional
-
ronin-16348 (at lighthouseapp)
- No changes were found…
-
ronin-16348 (at lighthouseapp)
I didn't see the attachment, just noticed where references it on the screen. I removed the duplicate attachment.
-
ronin-16348 (at lighthouseapp)
Forgot to add:
cattr_accessor :timestamped_migrations , :instance_writer => false
@@timestamped_migrations = false
This is required (as you guys know), in activerecord/lib/active_record/base.rb
Apologies, it's my first attempt at submitting a patch.
-
ronin-16348 (at lighthouseapp)
new diff file attached with active_record/base included.
-
Pratik
- State changed from new to incomplete
- Assigned user set to Pratik
- Tag set to patch
The default needs to be true.
Also, it'll be nice if you could send an email to rails core list to discuss this change.
Thanks.
-
ronin-16348 (at lighthouseapp)
I just sent an email to the core list. Is this why it was marked as incomplete?
thanks,
andy
-
nikz
Uploaded a diff that flips the default and adds docs, so us luddites can have our numbers back ;)
Now you kids get off my dang lawn!
-
Repository
- State changed from incomplete to resolved
(from [bbab6391366f59189e84d2b8de2a63bea91a9851]) Set config.active_record.timestamped_migrations = false to have migrations with numeric prefix instead of UTC timestamp. [#446 state:resolved]
Signed-off-by: Pratik Naik
-
ronin-16348 (at lighthouseapp)
Thanks nikz!
-
Repository
(from [af3f2aad7ecb31a6e1d09f06775cb33b4ad64d91]) Set config.active_record.timestamped_migrations = false to have migrations with numeric prefix instead of UTC timestamp. [#446 state:resolved]
Signed-off-by: Pratik Naik
