New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Update ReadMe bash script example #716

Merged

Slach merged 1 commit into Altinity:master from 0div:master

Aug 2, 2023

Contributor

0div commented Aug 2, 2023

Thanks for your awesome tool! I noticed some things to update in your example bash script:

Calling $? twice in a row has the effect of not capturing the first exit code wanted for logging, it is best to assign it to a var and preserve it that way
also reusing the capture exit code to exit the script with it (using 2>&1)
redirecting stderr to the log file as well.


          fixed ReadMe bash script example

97e460b

Slach merged commit 6eec986 into Altinity:master

7 of 16 checks passed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet