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.

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

Revise count docs #663

Merged
merged 4 commits into from
Jul 17, 2023
Merged

Revise count docs #663

merged 4 commits into from
Jul 17, 2023

Conversation

nwagner84
Copy link
Member

@nwagner84 nwagner84 commented Jul 15, 2023

@nwagner84 nwagner84 added C-documentation Category: documentation C-count labels Jul 15, 2023
@nwagner84 nwagner84 added this to the v1.0.0 milestone Jul 15, 2023
@nwagner84 nwagner84 self-assigned this Jul 15, 2023
Copy link
Contributor

@san-uh san-uh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Text ist prima. Bitte die optionalen Anmerkungen prüfen.

docs/book/src/referenz/kommandos/count.md Outdated Show resolved Hide resolved
docs/book/src/referenz/kommandos/count.md Outdated Show resolved Hide resolved
docs/book/src/referenz/kommandos/count.md Outdated Show resolved Hide resolved
@nwagner84 nwagner84 requested a review from san-uh July 17, 2023 06:37
Copy link
Contributor

@san-uh san-uh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Prima, ich denke auch, dass der Hinweis auf wc nicht zwingend ist. Imprimatur!

Copy link

@mschumac mschumac left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Das ist mal eine ausführliche Dokumentation, die aber angebracht erscheint. Inhaltlich gehe ich konform. Schau bitte noch mal "es wird keine Kopfzeile bei der CSV- bzw- TSV-Ausgabe geschrieben." an: "bzw." statt "bzw-". Dann bin ich mir nicht sicher, ob das sprachlich optimal formuliert ist. Wird keine Kopfzeile bei der Ausgabe geschrieben oder wird keine Kopfzeile in die Ausgabe geschrieben?

@nwagner84 nwagner84 requested a review from mschumac July 17, 2023 07:05
@nwagner84 nwagner84 merged commit 2c4b483 into main Jul 17, 2023
25 checks passed
@nwagner84 nwagner84 deleted the revise-count-docs branch July 17, 2023 07:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-documentation Category: documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants