@pablobm approved this pull request.

I can confirm this works, and makes a good argument to go for plural-named 
factory files as debated at 
https://github.com/openstreetmap/openstreetmap-website/pull/6607 (I'll get to 
that one).

With code as per `master`:
```
$ bin/rails g model doohicky
      invoke  active_record
      create    db/migrate/20251217150220_create_doohickies.rb
      create    app/models/doohicky.rb
      invoke    test_unit
      create      test/models/doohicky_test.rb
      create      test/fixtures/doohickies.yml
```

With code as per PR:

```
$ bin/rails g model doohicky
      invoke  active_record
      create    db/migrate/20251217150142_create_doohickies.rb
      create    app/models/doohicky.rb
      invoke    test_unit
      create      test/models/doohicky_test.rb
      invoke      factory_bot
      create        test/factories/doohickies.rb
```



-- 
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/6625#pullrequestreview-3588185800
You are receiving this because you are subscribed to this thread.

Message ID: 
<openstreetmap/openstreetmap-website/pull/6625/review/[email protected]>
_______________________________________________
rails-dev mailing list
[email protected]
https://lists.openstreetmap.org/listinfo/rails-dev

Reply via email to