safelyRenameFile errors do not exit with error code

[chrispsplash]

Describe the bug
If you get a safelyRenameFile error while using --inplace the process exits with status 0, indicating there was no error, without updating the file.

Version of yq: 4.16.2
Operating system: official docker container
Installed via: official docker container: mikefarah/yq:4.16.2

Reproduction
This Dockerfile (below 1) reproduces the error. Just build it and run to demonstrate (below 2)

Dockerfile

FROM mikefarah/yq:4.16.2

USER root

RUN echo -e 'foobar: baz' > /etc/tmp.yaml

USER yq

ENTRYPOINT []
CMD yq e '.foobar="hello"' -i /etc/tmp.yaml; echo exit code: $?

Build and run

docker build . -t yq-issue1062
docker run yq-issue1062

Actual behavior
Prints the error below but exits with code 0

22:50:44 safelyRenameFile [ERRO] Failed copying from /tmp/temp272450726 to /etc/tmp.yaml
22:50:44 safelyRenameFile [ERRO] open /etc/tmp.yaml: permission denied

Expected behavior
Prints the error above and exits with non-zero code

[mikefarah]

Thanks for the reproducible docker, this will be fixed in the next release.

[mikefarah]

Oh git autoclosed this :/ anyway the docker image 4.17.2 has the fix!