search

colin-scott / interactive_latencies

Jeff Dean's latency numbers plotted over time
MIT License
2.08k stars 455 forks source link

Mending a minor inconvenience. #21

Closed PranavPeshwe closed 1 year ago

PranavPeshwe commented 2 years ago

Removing the '$' in the documented curl command. When using the copy icon on the right, the dollar sign also gets pasted into the terminal.

SuperSandro2000 commented 2 years ago

It is pretty standard to indicate exectuable commands like this. a # would mean it should be run in a root shell.

PranavPeshwe commented 2 years ago

Agree 100% that it is standard. But IMO, the '$' hardly adds value. The context/rendering makes it clear that it is a command to be copy-pasted.

PranavPeshwe commented 1 year ago

Thanks, @port19x!