Skip to content

Commit 3b734a0

Browse files
committed
fix: fix review comments
Signed-off-by: Matthieu Gallien <matthieu.gallien@nextcloud.com>
1 parent 4d8095a commit 3b734a0

1 file changed

Lines changed: 7 additions & 7 deletions

File tree

AGENTS.md

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -49,19 +49,19 @@ Avoid creating source files that implement multiple types; instead, place each t
4949
- Include a short summary of what changed. *Example:* `fix: prevent crash on empty todo title`.
5050
- **Pull Request**: When the agent creates a PR, it should include a description summarizing the changes and why they were made. If a GitHub issue exists, reference it (e.g., “Closes #123”).
5151

52-
## c++ Specifics
52+
## C++ Specifics
5353

54-
The following details are important and only relevant when working on the desktop client parts written in c++ language.
54+
The following details are important and only relevant when working on the desktop client parts written in C++ language.
5555

56-
You should never use std::iostream for input or output handling and rather use Qt QLoggingCategory and related classes.
56+
You should never use `std::iostream` for input or output handling and rather use Qt's `QLoggingCategory` and related classes.
5757

58-
All logging categories starts with nextcloud identifier.
58+
All custom logging categories starts with our "nextcloud." prefix.
5959

60-
Our c++ code is using a pattern known as almost always auto when you should almost always not use explicit data types and rather use the auto keyword.
60+
Our C++ code is using a pattern known as almost always auto. You should almost never use explicit data types but the `auto` keyword for type declarations.
6161

62-
Our c++ code should can make use of c++ 20 standard features.
62+
Our C++ code should can make use of C++ 20 standard features whenever possible.
6363

64-
We do not use c++ modules and rather use standard include of headers.
64+
Do not use C++ modules. Use standard header inclusion instead.
6565

6666
## macOS Specifics
6767

0 commit comments

Comments
 (0)