search

drym-org / abe-sandbox

A prototype for a client repo using ABE accounting.
1 stars 0 forks source link

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

Open benknoble opened 1 year ago

benknoble commented 1 year ago

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 commented 1 year ago

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

benknoble commented 1 year ago

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.