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

Consider using $() instead of backticks for command substitution #41

Open
benknoble opened this issue Apr 26, 2023 · 2 comments
Open

Consider using $() instead of backticks for command substitution #41

benknoble opened this issue Apr 26, 2023 · 2 comments

Comments

@benknoble
Copy link

https://github.com/drym-org/abe-prototype-client/blob/f0ace1ff317f76741047c7f56ca45bcd42c9243e/.github/workflows/main.yml#L25

The backtick syntax foo is legacy and is not recommended.
@countvajhula
Copy link
Contributor

I blame @jairtrejo for this egregious use of non-POSIX compliant syntax 😆

@benknoble
Copy link
Author

Lol; I think it is POSIX. Just deprecated and recommended against.

Also funnily, the mobile GitHub client renders the back-quotes foo as a code-block-in-a-code-block.

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

No branches or pull requests
2 participants
@countvajhula @benknoble