Skip to content

Note to prefer $() over ``, and update single example to conform#100

Open
matthewcurry wants to merge 2 commits into
denysdovhan:masterfrom
matthewcurry:master
Open

Note to prefer $() over ``, and update single example to conform#100
matthewcurry wants to merge 2 commits into
denysdovhan:masterfrom
matthewcurry:master

Conversation

@matthewcurry
Copy link
Copy Markdown

Hello,

Most code style guides dictate that $() be used instead of `` for command substitution. I made a very small change to your document to encourage its use.

Thanks,
Matthew

@samumbach
Copy link
Copy Markdown
Collaborator

Nice catch, @matthewcurry 😃

I support this 💯 . We might want to swap the order of `` and $() in the prose and sample code block, too, just to reinforce that $() is more common (or should be... 😉 )

@matthewcurry
Copy link
Copy Markdown
Author

Hey, I added a commit to make the change you suggested.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants