Nicolas PHILIPPE
Nicolas PHILIPPE
even in max level, phpstan does not complain. But psalm is actually true: one param is missing in the methods prototype in the phpdoc
ok, this seems reasonable :) I feel the same, otherwise it would restrict too much further updates... see this issue as well https://github.com/zenstruck/foundry/issues/445#issuecomment-1500440459
this seems not possible because `ModelFactory::repository()` is static... I think this needs to be hardcoded directly in the class, and then have a factory class which is bound to a...
That's what I'm suggesting :)
Hi! global state + dama + `ResetDatabase` trait is quite a complex problem sometimes. I'm wondering if this is not the same problem than here: https://github.com/zenstruck/foundry/issues/534 (I'm running postgres, so...
> I have not tried to run tests without migrate because I don't think it is useful since Foundry uses DAMA if you are in migrate mode. It may work...
Maybe this part of the docs needs a reword: dama improves performance in both reset modes, since you only have to reset your db once. But it might improve drastically...
ok then, we've found the culprit: we actually have a problem with dama + reset_mode=migrate + global story @kbond I think we actually must add a full CI permutation with...
Hi @MrYamous This sounds good! I assume you'd want that only for nullable properties?
you're absolutely right! are you willing to provide a PR?