doc: MyFirstContribution: fix missing dependencies and clarify build steps
Fix issues in the MyFirstContribution guide that can lead to confusion or test failures when following the documented steps. * Add missing header includes in code examples (environment.h and strbuf.h). * Correct manpage synopsis formatting to prevent failing documentation tests. * Specify the use of parallel test execution with -j$(nproc), noting that it runs tests using all available CPUs and may be adjusted. These updates improve documentation accuracy and make the first-time contributor journey smoother. Signed-off-by: Shreyansh Paliwal <shreyanshpaliwalcmsmn@gmail.com> Signed-off-by: Junio C Hamano <gitster@pobox.com>
Shreyansh Paliwal committed
Jan 13, 2026 at 01:23 UTC
81021871eaa8b16a892b9c8791a0c905ab26e342
1 file changed
+9
-6
Documentation/MyFirstContribution.adoc
+9
-6
@@ -331,7 +331,8 @@ on the command line, including the name of our command. (If `prefix` is empty
331
for you, try `cd Documentation/ && ../bin-wrappers/git psuh`). That's not so
332
helpful. So what other context can we get?
333
334
-Add a line to `#include "config.h"` and `#include "repository.h"`.
334
+Add a line to `#include "config.h"`, `#include "repository.h"` and
335
+`#include "environment.h"`.
336
Then, add the following bits to the function body:
337
function body:
338
@@ -429,6 +430,7 @@ Add the following includes:
430
----
431
#include "commit.h"
432
#include "pretty.h"
433
+#include "strbuf.h"
434
----
435
436
Then, add the following lines within your implementation of `cmd_psuh()` near
@@ -503,8 +505,8 @@ git-psuh - Delight users' typo with a shy horse
505
506
SYNOPSIS
507
--------
506
-[verse]
507
-'git-psuh [<arg>...]'
508
+[synopsis]
509
+git psuh [<arg>...]
510
511
DESCRIPTION
512
-----------
@@ -726,9 +728,10 @@ $ prove -j$(nproc) --shuffle t[0-9]*.sh
728
----
729
730
NOTE: You can also do this with `make test` or use any testing harness which can
729
-speak TAP. `prove` can run concurrently. `shuffle` randomizes the order the
730
-tests are run in, which makes them resilient against unwanted inter-test
731
-dependencies. `prove` also makes the output nicer.
731
+speak TAP. `prove` can run concurrently. `-j$(nproc)` runs tests using all
732
+available CPUs in parallel, but the job count can be adjusted as needed.
733
+`shuffle` randomizes the order the tests are run in, which makes them resilient
734
+against unwanted inter-test dependencies. `prove` also makes the output nicer.
735
736
Go ahead and commit this change, as well.
737