-
-
Notifications
You must be signed in to change notification settings - Fork 554
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.
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
run shellcheck #97
run shellcheck #97
Conversation
Co-authored-by: Conrad Ludgate <oon@conradludgate.com>
I'm not sure why it's still not picking up the new action. Any ideas, @conradludgate ? |
D'oh! That's it! |
https://github.com/koalaman/shellcheck/wiki/SC1071, and the ignore: param in ludeeus/action-shellcheck only supports _directories_, not _files_. So instead, we manually add any error the shellcheck step finds in the file to the above line ...
Ready for review. One thing to put eyes on: this 92a74ea is fine, right? We're just checking for a status code there? |
Shellcheck + zsh: koalaman/shellcheck#809 |
Do you think you could add a comment to every ignore line? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for this! I've just got one small suggestion (as it looks like the = should not be there?)
Happy to merge once this is done 😊
Co-authored-by: Ellie Huxtable <ellie@elliehuxtable.com>
Thanks for catching that! I also resolved the merge conflict in install.sh. |
Thank you! 🚀 |
printf '\neval "$(atuin init zsh)"' >> ~/.zshrc | ||
|
||
curl https://raw.githubusercontent.com/rcaloras/bash-preexec/master/bash-preexec.sh -o ~/.bash-preexec.sh | ||
printf '\n[[ -f ~/.bash-preexec.sh ]] && source ~/.bash-preexec.sh' >> ~/.bashrc | ||
# Use of single quotes around $() is intentional here |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, they are. We want the string as is into the rc file.
Fixes #64