Tim de Jager

Results 280 comments of Tim de Jager

I could try making traits to get the required fields and make a minimal test like the one wolf suggested. But that would delay the merge a bit. Let me...

> It's not possible to make a vector and call sort on it in a test? I think you might think it was one of the trait implementations, which was...

I've made most of the changes, I'm not sure if the added complexity of the traits would be worth it. I could work on that if that is needed in...

> Perhaps we can make a simpler integration test by loading our sample repodata.json, sorting sprcific packages like libuuid and pytorch and creating some snapshots out of that? How is...

Thanks makes sense will do both. Let me make those test in main first in a separate PR, we can make sure the snapshots stay the same.

> One more final thing: Could you also run the rattler_solve benchmarks to see if we didn't accidentally introduce a performance regression? Ok, unfortunately, it seems to decrease performance by...

How would this as opposed to just installing `pixi global install gem` directly? I'd guess that you would have a version that's coupled to your other dependencies?

Yes makes sense but why do you want the ruby or nodejs from that specific environment exposed?

Well, I would just install `asciidoctor` seperately, from conda-forge. And if the ruby/gem versions are the same, it would just link it from the global package cache, and that would...

Haha ok yeah, I get it, so the thing is I think is that we do not know the exact location of the gem in this case, though we can...