Nate Wiebe

Results 30 comments of Nate Wiebe

Good call. Updated.

If we did want to use a simpler list, I like the format of this one: https://github.com/sass/node-sass#maintainers

This is likely caused by setting the ID strategy to AUTO, which was intended to make this more portable as Doctrine would then choose the appropriate method for setting the...

Thanks for your contribution. The v0 series is planned to only receive bugfixes moving forward. v1.0 includes a number of changes and would be best to use as a target...

I'll take a look at adding dbal 2 support back in. I can't remember the specifics, but I think there was some added complexity with supporting both. But considering Sylius...

> I think if any more configuration keys are added, it may be better to group them into arrayNodes and reduce the number of setter methods needed in AuditSubscriber, or...

Previously there wasn't a version constraint on `doctrine/dbal` which is likely the cause of this. Can you update to v0.1.21 and let us know if you still have this issue?

Correct, this was fixed in https://github.com/DATA-DOG/DataDogAuditBundle/commit/519d1574c44e5b889e509bd66c2200f400fb5dce. My guess here is the latest version of this bundle is not being used.

Also, it looks like there's duplication going on. `null` can likely be added to `T` and `D` can be removed completely

It looks like if I set the template on the function, rather than the class, it works as expected. Potentially this is a bug with Psalm itself.