Conversation
pks-t
left a comment
There was a problem hiding this comment.
Nice findings. Looks all good besides the missing @defgroup argument.
include/git2/transaction.h
Outdated
| /** | ||
| * @file git2/transaction.h | ||
| * @brief Git transaction routines. | ||
| * @defgroup Git transaction routines. |
There was a problem hiding this comment.
This should be @defgroup git_transaction Git transaction routines instead. Please also remove the trailing dots from @brief and @defgroup.
There was a problem hiding this comment.
We don't actually use these blocks for anything with our current documentation generator, so it should just go away.
There was a problem hiding this comment.
I was trying to make docurium document those functions. Wasn't sure if the group being defined was a prerequisite, but I'll take them out then.
|
Oh, one more thing. Could you maybe clarify the commit messages a bit by prefixing them with the module they're touching primaily, please? E.g. "remote: fix indentation". Thanks for your work 👍 |
include/git2/transaction.h
Outdated
| /** | ||
| * @file git2/transaction.h | ||
| * @brief Git transaction routines. | ||
| * @defgroup Git transaction routines. |
There was a problem hiding this comment.
We don't actually use these blocks for anything with our current documentation generator, so it should just go away.
| /** | ||
| * An opaque struct that stores the result of a describe operation. | ||
| */ | ||
| typedef struct git_describe_result git_describe_result; |
There was a problem hiding this comment.
The opaqueness is not something we need to specify in the text.
|
Patch series redone from scratch.
|
Note: 8149b17 removes git_remote_rename_problem_cb. Not sure when it stopped being used though.