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

Fix coloring in scripts #1305

Merged

qwerty287 merged 2 commits into master from fix-hook-color

May 7, 2022

Contributor

qwerty287 commented May 6, 2022

Must use printf instead of echo. Tested with bash and zsh.


          Fix coloring in scripts

codecov bot commented May 6, 2022 •

edited

Loading

Codecov Report

Merging #1305 (3905405) into master (50dacef) will decrease coverage by 0.89%.
The diff coverage is n/a.

❗ Current head 3905405 differs from pull request most recent head d598114. Consider uploading reports for the commit d598114 to get more accurate results

nagmat84 reviewed

View reviewed changes

scripts/install_files.sh Outdated Show resolved Hide resolved

nagmat84 approved these changes

View reviewed changes

Collaborator

nagmat84 left a comment

Thanks! That makes a lot of sense. I was constantly bothered by seeing ANSI control sequences in my terminal, but I did not feel summoned to look into it.


          Update scripts/install_files.sh

d598114

Co-authored-by: Matthias Nagel <matthias.h.nagel@posteo.de>

qwerty287 merged commit ddb1ddc into master

qwerty287 deleted the fix-hook-color branch

May 7, 2022 05:32

sonarqubecloud bot commented May 7, 2022

Kudos, SonarCloud Quality Gate passed!

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
No Duplication information

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

Labels

None yet