Matteo Ferrando

Results 95 comments of Matteo Ferrando

I understand. turning off the syscapture part maybe does not really make sense and this needs a different solution?

Although I can't think of where to put the directives to resolve a specific conflict.

We have a new strict check on dbt-core version because dbt-core 1.5.2 introduced breaking changes. We have to fix these before we can re open dbt-fal to 1.5.2+

I think the solution is not actually the best. This assumes there is a `virtuals` object to clone (which in turn assumes the [virtuals plugin](https://github.com/bookshelf/bookshelf/wiki/Plugin:-Virtuals) is enabled?). Wouldn't it make...

So it's for `belongsToMany` relations. But the test I see in this PR talks about a `belongsTo` relation, should it be changed to `belongsToMany` then?

Actually, I haven't reviewed it yet! I tried to understand the feature with the spec and since I'm not entirely sure how pivot tables work I can't really review it...

I think I see your point with the idea of passing in a single type of `object` or `object[]` and do the ORM to JSON transformation in another. It can...

Should they be added like `meta` attributes or as part of the normal `attributes`?