DEV: only enable SafeMigrate in development and test

DEV: only enable SafeMigrate in development and test

SafeMigrate outputs text when we detect attempts to unsafely drop tables and columns

It is unfortunately not thread safe

This is not needed in production as we would have already caught it by then in our test suite.

diff --git a/lib/freedom_patches/safe_migrations.rb b/lib/freedom_patches/safe_migrations.rb
index 718ffff..161d9fa 100644
--- a/lib/freedom_patches/safe_migrations.rb
+++ b/lib/freedom_patches/safe_migrations.rb
@@ -2,4 +2,11 @@
 
 require_dependency 'migration/safe_migrate'
 
-Migration::SafeMigrate.patch_active_record!
+# We do not run this in production cause it is intrusive and has
+# potential to break stuff, it also breaks under concurrent use
+# which rake:multisite_migrate uses
+#
+# The protection is only needed in Dev and Test
+if ENV['RAILS_ENV'] != "production"
+  Migration::SafeMigrate.patch_active_record!
+end

GitHub sha: 6b192d29