doc(readme): Add docker usage doc #3103
Conversation
|
This is nice improvement. |
Co-authored-by: itchyny <[email protected]>
|
Thank you |
| ``` | ||
| ##### Example: Extracting the version from a `package.json` file with a mounted volume | ||
| ```bash | ||
| docker run --rm -i -v "${PWD}":/pwd -w /pwd ghcr.io/jqlang/jq:latest '.version' package.json |
There was a problem hiding this comment.
One possible improvement could be to do -v "$PWD:$PWD" -w "$PWD" to make paths work more as expected
There was a problem hiding this comment.
Not sure I understand, Can you maybe share an example command that will act in an unexpected way?
There was a problem hiding this comment.
If one would happen to use absolute host paths in arguments etc they won't work, ex:
docker run --rm -i -v "${PWD}":/pwd -w /pwd ghcr.io/jqlang/jq:latest -f $PWD/file.jq ...but even with the -v -w trick above paths outside host $PWD will not work... so not perfect either.
There was a problem hiding this comment.
I think this is a slight difference no need to fix. -f accepts relative path, env.PWD is different, but should be ok.
There was a problem hiding this comment.
Opened a PR #3105
There was a problem hiding this comment.
@itchyny yeap not big thing but it has hit me a couple of times before :)
Hey, added a short doc on running the tool inline with docker