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 cat documentation #657

Merged
merged 9 commits into from
Jul 13, 2023
Merged

Revise cat documentation #657

merged 9 commits into from
Jul 13, 2023

Conversation

nwagner84
Copy link
Member

@nwagner84 nwagner84 commented Jul 12, 2023

Link zur neuen Dokumentation (Review-Modus): https://github.com/deutsche-nationalbibliothek/pica-rs/pull/657/files#diff-456d834ac51aca9e7e1220b1d15ee3df946cfa3576c74cb04ab1d9fbe34e131a

Link zum besseren Ansicht: https://github.com/deutsche-nationalbibliothek/pica-rs/blob/0c830c3b604370a4cac50ae79243e62aa6c8098d/docs/book/src/referenz/kommandos/cat.md

Schritte zum Durchführen des Review:

  1. Neue Doku im Review-Modus öffnen
  2. Kommentare über "+"-Button hinzufügen, falls etwas auffällt
  3. Abschließen des Reviews ("Finish your review", oben rechts) (Approve = alles ok, Comment = allgemeiner Kommentar, Request Change = Änderungsvorschläge)

Kleinere Schreibfehler korrigiert.
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.

Gelesen, koorigiert. Kann online gehen!

@nwagner84 nwagner84 added C-documentation Category: documentation C-cat labels Jul 12, 2023
@nwagner84 nwagner84 marked this pull request as ready for review July 12, 2023 19:23
Copy link

@LMoelck LMoelck left a comment

Choose a reason for hiding this comment

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

Hi Nico,
irgendetwas habe ich falsch gemacht, da mir die Option "Finish your review" nicht angezeigt wird.
Ich hoffe, Du kannst trotzdem die Kommentare lesen.
Beim nächsten Mal klappt es mit meiner Bearbeitung bestimmt besser.
Auf jeden Fall freue ich mich schon jetzt, nach meinem Urlaub weiter zu machen:-)

Viele Grüße, Letitia

@nwagner84
Copy link
Member Author

Liebe @LMoelck , Lieber @mschumac , Lieber @san-uh ,

vielen Dank für das Review! Die neue Version wird dann gleich online gehen!

@nwagner84 nwagner84 merged commit cb87bde into main Jul 13, 2023
26 checks passed
@nwagner84 nwagner84 deleted the revise-cat-docs branch July 13, 2023 12:40
nwagner84 added a commit that referenced this pull request Jul 13, 2023
nwagner84 added a commit that referenced this pull request Jul 13, 2023
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.

4 participants