Georg Semmler

@valkrypton As already pointed out: `array_sample` is a relatively new function, so you cannot use it unless you installed postgres 16. I would go with the same solution as in...

@valkrypton I do not have such an idea yet. That would require doing some basic design work first. I don't have the capacity for doing this now.

@valkrypton Set types are kind of special. They don't behave like for example arrays, but more like a table. So you can use them in your from clause and also...

> Only thing I could not figure out was how to add the tests to CI. The tests are automatically picked up: https://github.com/diesel-rs/diesel/actions/runs/10411801607/job/28836874448?pr=4169#step:27:356. I'm not sure why they fail but...

Thanks for the update :+1: I've rebased the PR to resove the merge conflicts. I'm now going over the changes to perform another review.

Thanks for the update. I will try to do a final review (and likely some additional minor adjustments based on that) this week. Otherwise I think this is fine now....

> While things worked locally on my box, apparently on CI each test really needs an unique connection. The problem there was that you likely had already applied the migrations...

> As for the guide, I don't have the capacity to do that as my project progresses, but it might be an option to open an issue with a call...

@bcspragu I took some time to add support for multiple gitattributes files on top of your change in https://github.com/weiznich/jj/commit/a12505164ac655c8769dfee9991aae5ccf20990f. That also fixes the problem with the relative paths. Maybe one...

@martinvonz https://github.com/weiznich/jj/blob/ignore_lfs/lib/src/matchers.rs#L592-L616 implements exactly that. I've run it locally on a repository that useses several gitattributes files in different subfolders and it seems to work fine.