Reid Priedhorsky

Results 153 comments of Reid Priedhorsky

### Superclass argument renamed (W0237 arguments-renamed) - [ ] done https://github.com/hpc/charliecloud/blob/d67937332094a7d8de8e9bd2e112d30d117c50c0/lib/charliecloud.py#L286 Personally I think `value` is a bad name but this is probably right, less confusing if we use the...

### Method that only delegates back to parent (W0246 useless-parent-delegation) - [ ] done https://github.com/hpc/charliecloud/blob/d67937332094a7d8de8e9bd2e112d30d117c50c0/lib/image.py#L928-L929 Indeed this method seems to be doing nothing. Solution is probably delete it regardless, but...

### Bad indentation (W0311 bad-indentation) - [ ] done https://github.com/hpc/charliecloud/blob/d67937332094a7d8de8e9bd2e112d30d117c50c0/lib/build_cache.py#L827-L829 This whole method is indented too much, probably due to being moved at some point.

@rarensu raised a use case in discussion #1562 that we hadn't thought of: > Docker, Podman, and Singularity all have inspect features with varying utility. The most important use of...

This has been coming up at least somewhat frequently lately, so to clarify the status: yes I've been convinced we should do this. Hopefully soon.

e.g. [this one](https://github.com/json-parser/json-parser) is BSD licensed and recommends “just drop `json.c` and `json.h` into your project”.

[this one](https://github.com/DaveGamble/cJSON) similarly

@j-ogas do you have an opinion on whether it lives in `ch-run` or a Python wrapper?

I think this would be cool but likely a lower priority. Do Docker and Podman do this?

Sorry, clicked "close with comment" by mistake.