Ben Mares comments

Results 675 comments of


                                            Ben Mares

mambauser cannot write to bind mounts on Linux with Docker in rootless-mode

Wow, thanks @mfhepp for your persistence in getting to the bottom of this issue! This looks like it was quite some effort. I have not yet used rootless mode myself...

Fix narrowing errors by replacing int by ssize_t

Perhaps, I don't really understand much myself.

Fix narrowing errors by replacing int by ssize_t

Great, thanks a lot @aseyboldt for the "pointer" to `ptrdiff_t`. :slightly_smiling_face: I'm busy with some other stuff at the moment, but I'll come back to this. I'd like to enable...

Fix narrowing errors by replacing int by ssize_t

Was hoping for an easy way out with https://github.com/conda-forge/sunode-feedstock/pull/35, but it didn't work.

Add deptry to check for issues with the project's dependencies.

This looks really interesting but I'm slammed at the moment. Could you ping me in a few weeks so I can give this a proper look?

Add deptry to check for issues with the project's dependencies.

Thanks a lot @fpgmaas, this is a really cool project. I like that it pushed me to document several quirks of our dependencies in the ignores. Moreover I opened #812...

Add deptry to check for issues with the project's dependencies.

Squished.

Add deptry to check for issues with the project's dependencies.

Just realized that currently deptry only exists for Linux and AMD Mac on conda-forge. That might break the workflows of developers. Will remove it from the environment.yaml

Fix E721: do not compare types, for exact checks use is / is not

@ricardoV94, rebase is complete.

Fix E721: do not compare types, for exact checks use is / is not

> @maresb were we suppressing this pre-commit rule, should we reactivate it? No we weren't, it's just totally broken in Ruff: https://github.com/astral-sh/ruff/issues/6465 But now E721 passes with flake8