-
Notifications
You must be signed in to change notification settings - Fork 175
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
Remove every bashism #282
Remove every bashism #282
Conversation
c5b8ea3
to
9172083
Compare
https://mywiki.wooledge.org/Bashism Here is the site I used to squeeze out these bashisms. Might be helpful to Chris in future development. |
2362c89
to
a7073ca
Compare
b28d2b2
to
3a0d0ce
Compare
Just squeezed out the last bashisms out of dwm-titus. Ready for review / merge. |
Sorry for the inconvenience. We had a massive restructure of the codebase to improve future development. Because of this can you update your PR to the new structure. Thank you for your assistance and contribution. |
Finished. |
Thanks so much looks awesome @nnyyxxxx |
Type of Change
Description
I replaced every bashism with posix alternatives, I removed the custom color codes that were in almost every file (utilities), I refactored a lot of stuff (utilities), and I made it rely on the common script color codes instead of the custom ones (utilities).
Testing
Tested on bash, zsh, and dash.
Rechecked every file using
checkbashisms
packageImpact
Greatly impacts the changed sections.
Issue related to PR
Checklist