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.

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

(SH-2001) Consider using ${variable//search/replace} #16

Closed
johnsonjh opened this issue Apr 17, 2021 · 2 comments
Closed

(SH-2001) Consider using ${variable//search/replace} #16

johnsonjh opened this issue Apr 17, 2021 · 2 comments
Labels
wontfix This will not be worked on

Comments

@johnsonjh
Copy link
Owner

Description

Consider using parameter expansion to search and replace data. In the problematic code the value is passed to sed to do this, but for simple substitutions, parameter expansion can do it with less overhead. #### Problematic code: bash string="stirng" ; echo "$string" | sed -e "s/ir/ri/" #### Preferred code: …

Occurrences

There are 5 occurrences of this issue in the repository.

See all occurrences on DeepSource → deepsource.io/gh/johnsonjh/sgs-68k/issue/SH-2001/occurrences/
@johnsonjh
Copy link
Owner Author

Won't fix / legacy / historical

@johnsonjh johnsonjh added the wontfix This will not be worked on label Apr 17, 2021
@repo-ranger
Copy link

repo-ranger bot commented Apr 17, 2021

⚠️ This has been marked to be closed in 7 days.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
wontfix This will not be worked on
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@johnsonjh