Re: [PATCH] t: fix typos

From: Kristoffer Haugsbakk <kristofferhaugsbakk@fastmail.com>
Date: Oct 23, 2024, 18:00 UTC
Message-ID: <fa5d771b-5819-4ede-bc95-9d50c456c619@app.fastmail.com>
In-Reply-To: <20241023170111.6638-1-algonell@gmail.com>

All of the changes here look good.

On Wed, Oct 23, 2024, at 19:01, Andrew Kreimer wrote:

Show 18 quoted lines

> Fix typos in documentation, comments, etc.
>
> Via codespell.
>
> Signed-off-by: Andrew Kreimer <algonell@gmail.com>
> […]
> diff --git a/t/t5303-pack-corruption-resilience.sh
> b/t/t5303-pack-corruption-resilience.sh
> index e6a43ec9ae..b634bfb665 100755
> --- a/t/t5303-pack-corruption-resilience.sh
> +++ b/t/t5303-pack-corruption-resilience.sh
> @@ -15,7 +15,7 @@ TEST_PASSES_SANITIZE_LEAK=true
>  # 1) blob_2 is a delta with blob_1 for base and blob_3 is a delta with
> blob2
>  #    for base, such that blob_3 delta depth is 2;
>  #
> -# 2) the bulk of object data is uncompressible so the text part remains
> +# 2) the bulk of object data is incompressible so the text part remains

Unusual word IME but makes sense here after googling: https://www.merriam-webster.com/dictionary/contiguous

Show 13 quoted lines

> […]
> diff --git a/t/t5528-push-default.sh b/t/t5528-push-default.sh
> index bc2bada34c..aecd03d69f 100755
> --- a/t/t5528-push-default.sh
> +++ b/t/t5528-push-default.sh
> @@ -147,7 +147,7 @@ test_expect_success 'push from/to new branch fails
> with upstream and simple ' '
>  #  - the default push succeeds
>  #
>  # A previous test expected this to fail, but for the wrong reasons:
> -# it expected a fail becaause the branch is new and cannot be pushed,
> but
> +# it expected a fail because the branch is new and cannot be pushed,

Also: s/a fail/a failure/ ?

Perhaps “to fail”.

Show 12 quoted lines

> […]
> diff --git a/t/t5558-clone-bundle-uri.sh b/t/t5558-clone-bundle-uri.sh
> index cd05321e17..3816ed5058 100755
> --- a/t/t5558-clone-bundle-uri.sh
> +++ b/t/t5558-clone-bundle-uri.sh
> @@ -945,7 +945,7 @@ test_expect_success 'creationToken heuristic with
> failed downloads (clone)' '
>  		--bundle-uri="$HTTPD_URL/bundle-list" \
>  		"$HTTPD_URL/smart/fetch.git" download-3 &&
>
> -	# As long as we have continguous successful downloads,
> +	# As long as we have contiguous successful downloads,

Contiguous in the sense of “next or near in time or sequence”. Makes sense in this context.

Show 12 quoted lines

> […]
> diff --git a/t/t7064-wtstatus-pv2.sh b/t/t7064-wtstatus-pv2.sh
> index 06c1301222..2458e9d0eb 100755
> --- a/t/t7064-wtstatus-pv2.sh
> +++ b/t/t7064-wtstatus-pv2.sh
> @@ -77,7 +77,7 @@ test_expect_success 'before initial commit, things
> added (-z)' '
>  	test_cmp expect actual
>  '
>
> -test_expect_success 'make first commit, comfirm HEAD oid and branch' '
> +test_expect_success 'make first commit, confirm HEAD oid and branch' '

Easy to mistake m/n in monospace font. :P

> […]