Not copying Folio factories to main app, just add them to lookup #143
Conversation
…nt; small improvement in mailer deprecation check
…nt; small improvement in mailer deprecation check
foton
changed the title
foton
changed the title
There was a problem hiding this comment.
Nerozumim motivaci pro cely tenhle PR. Co je spatne na tom, ze se volaji folio factories ve factories souboru v projektu? Tech pripadu tam muze byt vic, napr.
require_relative Folio::Engine.root.join("test/factories")
require_relative Boutique::Engine.root.join("test/factories")
| require 'factory_bot_rails' | ||
| rescue LoadError | ||
| end | ||
|
|
There was a problem hiding this comment.
Proc begin/rescue a proc ne jen v test env?
There was a problem hiding this comment.
Ne v každém projektu nutně máme factory_bot_rails
There was a problem hiding this comment.
Například Folio mělo do teď jen factory_bot (viz změny v gemspecu)
There was a problem hiding this comment.
V kazdem projektu s foliem a s factories ale ano, nebo ne?
| @@ -87,7 +87,6 @@ def copy_templates | |||
| "db/migrate/20220214083648_rm_private_attachments_mime_type_column.rb", | |||
| "lib/tasks/developer_tools.rake", | |||
| "public/maintenance.html", | |||
| "test/factories.rb", | |||
| "test/test_helper.rb", | |||
There was a problem hiding this comment.
@foton Takhle tam ale ten soubor vubec nevznikne. Soubor tam chceme.
There was a problem hiding this comment.
Ano, to je to kopírování,, které se snažím nahradit linkováním.
There was a problem hiding this comment.
Takhle to ale zustalo v templates. Radeji nez vyhazovat odsud bych upravil ten soubor. Abych ho nemusel v novem projektu zakladat.
Oprava pár drobností na které jsem narazil při vývoji a
vypuštění kopírování factories do hlavní aplikace. Místo toho se jen přidají (z Folia) do cest pro hledání factories.
Takže zůstanou tam kam patří (Folio), ale dají se použít i z hlavní aplikace (resp jejích testů).