Skip to content
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

Fix minor ShellCheck violations #537

Merged

Conversation

hyperupcall
Copy link
Contributor

@hyperupcall hyperupcall commented Nov 5, 2023

  • Replace deprecated backticks with $()
  • Replace use of deprecated egrep with grep -E
  • Disable ShellCheck false positive
  • Every single use of gp_set_window_title has 1 positional parameter, so only interpolate $1

This is a good project, and I noticed that there were several issues that have already been fixed, or are old enough that they are not relevant (uses an EOL Linux Distribution). I have marked the following as to close. There are more, but I think this is a good start. Closing these old issues makes the repository seem more maintained, and it's easier to triage and look for existing issues when they are closed.

Closes #420 (stale, not enough information)
Closes #327 (stale, fixed on later Debian version)
Closes #225 (stale, fixed on later Ubuntu version)
Closes #347 (stale, fixed in later bash-git-prompt version)
Closes #440 (stale, could not reproduce, / fixed in later bash-git-prompt version)
Closes #494 (stale, not relevant)
Closes #291 (stale)
Closes #531 (duplicate of #466)
Closes #533 (stale / spam / bot)

@rakotomandimby rakotomandimby merged commit ec14bec into magicmonty:master Nov 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants