You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I have a situation where I have two recipe dependencies which both have the same project file:
silverstripe/recipe-core: app/_config.php
cwp/cwp-recipe-core: app/_config.php
Because app/_config.php from silverstripe/recipe-core is run before pretty much anything else (as a close-to-root dependency), it takes the project file from there and copies it into my project. Subsequent recipes such as cwp/cwp-recipe-core cannot copy their own file after this, because they get ignore as "existing or modified in project."
The problem in this example is really that the file contains password complexity requirements, and because they are then copied into user code they will take priority over everything else. CWP needs to increase the minimum requirements but it can't do so at the moment.
The text was updated successfully, but these errors were encountered:
FWIW I'm proposing that we remove this project file from recipe-core and move it into YAML configuration instead, which is extensible. This issue could serve as an enhancement request to allow controlling priority between recipes that have the same project files in them though.
I have a situation where I have two recipe dependencies which both have the same project file:
Because
app/_config.php
from silverstripe/recipe-core is run before pretty much anything else (as a close-to-root dependency), it takes the project file from there and copies it into my project. Subsequent recipes such as cwp/cwp-recipe-core cannot copy their own file after this, because they get ignore as "existing or modified in project."The problem in this example is really that the file contains password complexity requirements, and because they are then copied into user code they will take priority over everything else. CWP needs to increase the minimum requirements but it can't do so at the moment.
The text was updated successfully, but these errors were encountered: