Skip to content

Commit 4d8095a

Browse files
committed
feat(agents): add some more details on what we want for c++ code
Signed-off-by: Matthieu Gallien <matthieu.gallien@nextcloud.com>
1 parent 51a1981 commit 4d8095a

1 file changed

Lines changed: 14 additions & 0 deletions

File tree

AGENTS.md

Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -49,6 +49,20 @@ 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
53+
54+
The following details are important and only relevant when working on the desktop client parts written in c++ language.
55+
56+
You should never use std::iostream for input or output handling and rather use Qt QLoggingCategory and related classes.
57+
58+
All logging categories starts with nextcloud identifier.
59+
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.
61+
62+
Our c++ code should can make use of c++ 20 standard features.
63+
64+
We do not use c++ modules and rather use standard include of headers.
65+
5266
## macOS Specifics
5367

5468
The following details are important and only relevant when working on the desktop client on macOS.

0 commit comments

Comments
 (0)