-
-
Notifications
You must be signed in to change notification settings - Fork 4.8k
*: remove non-ASCII characters where appropriate #5768
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.
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
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change | ||||
---|---|---|---|---|---|---|
|
@@ -18,8 +18,8 @@ | |||||
|
||||||
- Aggiungi uno o più ebook alla libreria: | ||||||
|
||||||
`calibredb add {{file1 file2 …}}` | ||||||
`calibredb add {{file1}} {{file2}}` | ||||||
|
||||||
- Rimuovi uno o più ebook dalla libreria. Sono necessari gli ID (vedi sopra): | ||||||
|
||||||
`calibredb remove {{id1 id2 …}}` | ||||||
`calibredb remove {{id1}} {{id2}}` | ||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Same here
Suggested change
|
Original file line number | Diff line number | Diff line change | ||||
---|---|---|---|---|---|---|
|
@@ -26,7 +26,7 @@ | |||||
|
||||||
- Scan a specific list of ports (use -p- for all ports 1-65535): | ||||||
|
||||||
`nmap -p {{port1,port2,…,portN}} {{address_or_addresses}}` | ||||||
`nmap -p {{port1,port2,port3}} {{address_or_addresses}}` | ||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. We should keep the portN here at least.
Suggested change
|
||||||
|
||||||
- Perform TCP and UDP scanning (use -sU for UDP only, -sZ for SCTP, -sO for IP): | ||||||
|
||||||
|
Original file line number | Diff line number | Diff line change | ||||
---|---|---|---|---|---|---|
|
@@ -5,16 +5,16 @@ | |||||
- Check pathames for validity in the current system: | ||||||
|
||||||
`pathchk {{path1 path2 …}}` | ||||||
`pathchk {{path1}} {{path2}}` | ||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. It almost looks like the original is standard syntax for multiple options (which I'm a fan of 👍🏻). If it is, I'd like to open a new PR to standardize this across the other pages (and I'll fix it here as well). Is it? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I like it too. It should render to the exact same result and looks cleaner. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Well, not exactly the same, depending on your client. Mine, the node client, underlines text in token syntax, so the difference between the old one (which I prefer) and the new modification I made (which I'll revert) is that the old one underlines the space because the space is included between the handlebars, while in the newer version, the space is outside of them. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Generally in other pages I've done I think it's dependent on context |
||||||
|
||||||
- Check pathnames for validity on a wider range of POSIX compliant systems: | ||||||
|
||||||
`pathchk -p {{path1 path2 …}}` | ||||||
`pathchk -p {{path1}} {{path2}}` | ||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This can be an arbitrary number of items, which this edit is not expressing.
Suggested change
|
||||||
|
||||||
- Check pathnames for validity on all POSIX compliant systems: | ||||||
|
||||||
`pathchk --portability {{path1 path2 …}}` | ||||||
`pathchk --portability {{path1}} {{path2}}` | ||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Same here
Suggested change
|
||||||
|
||||||
- Only check for empty pathnames or leading dashes (-): | ||||||
- Only check for empty pathnames or leading dashes: | ||||||
|
||||||
`pathchk -P {{path1 path2 …}}` | ||||||
`pathchk -P {{path1}} {{path2}}` | ||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. same here
Suggested change
|
Original file line number | Diff line number | Diff line change | ||||
---|---|---|---|---|---|---|
|
@@ -2,6 +2,7 @@ | |||||
|
||||||
> Tools for handling high-throughput sequencing (genomics) data. | ||||||
> Used for reading/writing/editing/indexing/viewing of data in SAM/BAM/CRAM format. | ||||||
> More information: <https://www.htslib.org>. | ||||||
- Convert a SAM input file to BAM stream and save to file: | ||||||
|
||||||
|
@@ -15,7 +16,7 @@ | |||||
|
||||||
`samtools sort {{input}} -o {{output.bam}}` | ||||||
|
||||||
- Index a sorted BAM file (creates {{sorted_input.bam.bai}}): | ||||||
- Index a sorted BAM file (creates `sorted_input.bam.bai`): | ||||||
|
||||||
`samtools index {{sorted_input.bam}}` | ||||||
|
||||||
|
@@ -29,7 +30,7 @@ | |||||
|
||||||
- Merge multiple files: | ||||||
|
||||||
`samtools merge {{output}} {{input1 input2 …}}` | ||||||
`samtools merge {{output}} {{input1}} {{input2}}` | ||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Again with the changing the meaning.
Suggested change
|
||||||
|
||||||
- Split input file according to read groups: | ||||||
|
||||||
|
Original file line number | Diff line number | Diff line change | ||||
---|---|---|---|---|---|---|
|
@@ -13,7 +13,7 @@ | |||||
|
||||||
- Show the history of specific files or directories: | ||||||
|
||||||
`tig {{path1 path2 …}}` | ||||||
`tig {{path1}} {{path2}}` | ||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||||
|
||||||
- Show the difference between two references (such as branches or tags): | ||||||
|
||||||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This can be an arbitrary number of items.
Uh oh!
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Would it be possible to replace
path/to/file
withfile
in general? It looks much cleaner and readable and in most cases it's redundent. In cases where it's important we could keep it.Uh oh!
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm a fan of
path/to/file
because it clearly indicates that it's a file path, as opposed to a file name (because there are commands that take a file name, not path, e.g. this). It also makes it easy to indicate if the path must be an absolute one, which, in fact, many tools require.Also,
file
can look more like a variable in many contexts, whereaspath/to/file
is very clearly one and only one thing.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
path/to/file
is our convention elsewhere, so generally speaking we prefer it overfile
. Butfile
can be useful if you have many items, sure as here.