[openstreetmap/openstreetmap-website] Trace import: remove activerecord-import gem (PR #5038)
mmd
notifications at github.com
Sun Aug 4 12:37:52 UTC 2024
@mmd-osm commented on this pull request.
> @@ -71,6 +71,7 @@ Rails/SkipsModelValidations:
Exclude:
- 'db/migrate/*.rb'
- 'app/controllers/users_controller.rb'
+ - 'app/models/trace.rb'
Rubocop was complaining about the use of insert_all! because it's bypassing validations.
--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/5038#discussion_r1703177073
You are receiving this because you are subscribed to this thread.
Message ID: <openstreetmap/openstreetmap-website/pull/5038/review/2217742832 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20240804/a5915308/attachment.htm>
More information about the rails-dev
mailing list