<feed xmlns='http://www.w3.org/2005/Atom'>
<title>git/.github, branch jch</title>
<subtitle>Mirror of https://git.kernel.org/pub/scm/git/git.git/
</subtitle>
<id>https://git.shady.money/git/atom?h=jch</id>
<link rel='self' href='https://git.shady.money/git/atom?h=jch'/>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/'/>
<updated>2026-04-15T21:31:40Z</updated>
<entry>
<title>codeql: bump actions/cache from 4 to 5</title>
<updated>2026-04-15T21:31:40Z</updated>
<author>
<name>Johannes Schindelin</name>
<email>johannes.schindelin@gmx.de</email>
</author>
<published>2026-04-13T16:24:17Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=f65aba1e87db64413b6d1ed5ae5a45b5a84a0997'/>
<id>urn:sha1:f65aba1e87db64413b6d1ed5ae5a45b5a84a0997</id>
<content type='text'>
Bumps [actions/cache](https://github.com/actions/cache) from 4 to 5.
- [Release notes](https://github.com/actions/cache/releases)
- [Changelog](https://github.com/actions/cache/blob/main/RELEASES.md)
- [Commits](https://github.com/actions/cache/compare/v4...v5)

updated-dependencies:
- dependency-name: actions/cache
  dependency-version: '5'
  dependency-type: direct:production
  update-type: version-update:semver-major

Originally-authored-by: dependabot[bot] &lt;support@github.com&gt;
Signed-off-by: Johannes Schindelin &lt;johannes.schindelin@gmx.de&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
<entry>
<title>Merge branch 'jc/ci-github-actions-use-checkout-v5'</title>
<updated>2026-04-13T20:54:57Z</updated>
<author>
<name>Junio C Hamano</name>
<email>gitster@pobox.com</email>
</author>
<published>2026-04-13T20:54:57Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=7a5d03c93c71dd6bbf607f5d07eda665e3826282'/>
<id>urn:sha1:7a5d03c93c71dd6bbf607f5d07eda665e3826282</id>
<content type='text'>
CI dependency updates.

* jc/ci-github-actions-use-checkout-v5:
  CI: bump actions/checkout from 4 to 5 for rust-analysis job
</content>
</entry>
<entry>
<title>CI: bump actions/checkout from 4 to 5 for rust-analysis job</title>
<updated>2026-04-13T18:35:31Z</updated>
<author>
<name>Junio C Hamano</name>
<email>gitster@pobox.com</email>
</author>
<published>2026-04-13T18:24:44Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=4bdb17e3a8767195cc0540a6336bfd22d4b6445a'/>
<id>urn:sha1:4bdb17e3a8767195cc0540a6336bfd22d4b6445a</id>
<content type='text'>
GitHub Actions started complaining about use of Node.js 20 and I was
wondering why only one job uses actions/checkout@v4, while everybody
else already uses actions/checkout@v5.

It turns out that it is caused by a semantic mismerge between
e75cd059 (ci: check formatting of our Rust code, 2025-10-15) that
added a new use of actions/checkout@v4 that happened very close to
another change 63541ed9 (build(deps): bump actions/checkout from 4
to 5, 2025-10-16) that updated all uses of actions/checkout@v4 to
use vactions/checkout@v5.

Update the leftover and the last use of actions/checkout@v4 to use
actions/checkout@v5 to help ourselves to move away from Node.js 20.

Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
<entry>
<title>Merge branch 'ps/ci-gitlab-msvc-updates'</title>
<updated>2026-02-27T23:11:52Z</updated>
<author>
<name>Junio C Hamano</name>
<email>gitster@pobox.com</email>
</author>
<published>2026-02-27T23:11:52Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=a62d0da86a763ab03604d75eb287776a7d639e1f'/>
<id>urn:sha1:a62d0da86a763ab03604d75eb287776a7d639e1f</id>
<content type='text'>
CI update.

* ps/ci-gitlab-msvc-updates:
  gitlab-ci: handle failed tests on MSVC+Meson job
  gitlab-ci: use "run-test-slice-meson.sh"
  ci: make test slicing consistent across Meson/Make
  github: fix Meson tests not executing at all
  meson: fix MERGE_TOOL_DIR with "--no-bin-wrappers"
  ci: don't skip smallest test slice in GitLab
  ci: handle failures of test-slice helper
</content>
</entry>
<entry>
<title>ci: make test slicing consistent across Meson/Make</title>
<updated>2026-02-19T18:22:31Z</updated>
<author>
<name>Patrick Steinhardt</name>
<email>ps@pks.im</email>
</author>
<published>2026-02-19T06:25:31Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=2cca4ef34306c04a8ae891e0ea4a876922069dce'/>
<id>urn:sha1:2cca4ef34306c04a8ae891e0ea4a876922069dce</id>
<content type='text'>
In the preceding commit we have adjusted test slicing to be one-based
when using the "ci/run-test-slice.sh" script. But we also have an
equivalent script for Meson that is still zero-based, which is of course
inconsistent.

Adapt the script to be one-based, as well, and adapt the GitHub workflow
accordingly. Note that GitLab doesn't yet use the script, so it does not
need to be adapted. This will change in the next commit though.

Signed-off-by: Patrick Steinhardt &lt;ps@pks.im&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
<entry>
<title>github: fix Meson tests not executing at all</title>
<updated>2026-02-19T18:22:31Z</updated>
<author>
<name>Patrick Steinhardt</name>
<email>ps@pks.im</email>
</author>
<published>2026-02-19T06:25:30Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=420b4ca2dce7143fcd79892393a879a1d8e276f7'/>
<id>urn:sha1:420b4ca2dce7143fcd79892393a879a1d8e276f7</id>
<content type='text'>
While the win+Meson test jobs run in GitHub workflows, the shell script
that is supposed to run the jobs is seemingly not running at all. All
that the CI job prints is the following:

  Run ci/run-test-slice-meson.sh build 1 10
    ci/run-test-slice-meson.sh build 1 10
    shell: C:\Program Files\PowerShell\7\pwsh.EXE -command ". '{0}'"
    env:
      DEVELOPER: 1

The step is currently defined to use PowerShell, and of course it
doesn't know how to execute POSIX shell scripts. What's surprising
though is that this step doesn't even lead to a CI failure.

Fix the issue by using Bash instead of PowerShell, as we do in other
steps that execute shell scripts.

Signed-off-by: Patrick Steinhardt &lt;ps@pks.im&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
<entry>
<title>ci: don't skip smallest test slice in GitLab</title>
<updated>2026-02-19T18:22:30Z</updated>
<author>
<name>Patrick Steinhardt</name>
<email>ps@pks.im</email>
</author>
<published>2026-02-19T06:25:28Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=3141df7ec44f5f2579fd839a6b5a457df109c5e6'/>
<id>urn:sha1:3141df7ec44f5f2579fd839a6b5a457df109c5e6</id>
<content type='text'>
The "ci/run-test-slice.sh" script can be used to slice up all of our
tests into N pieces and then run each of them on a separate CI job.
This is used by both GitLab and GitHub CI to speed up Windows tests,
which would otherwise be painfully slow.

The infra itself is fueled by `test-tool path-utils slice-tests`. This
tool receives as input an "offset" and a "stride" that can be combined
to slice up tests. This framing can be misleading though: you are
expected to pass a zero-based index as "offset", and the complete number
of slices to the "stride". The latter makes sense, but it is somewhat
surprising that the offset needs to be zero-based. And this is in fact
biting us: while GitHub passes zero-based indices, GitLab passes
`$CI_NODE_INDEX`, which is a one-based indice.

Ideally, we should have verification that the parameters make sense.
And naturally, one would for example expect that it's an error to call
the binary with an offset larger than the stride. But with the current
framing as "offset" it's not even wrong to do so, as it is of course
well-defined to start at a larger offset than the stride.

This means that we get this wrong on GitLab's CI, as we pass a one based
index there, and this causes us to skip one of the tests. Interestingly,
it's not the lexicographically first test that we skip. Instead, as we
sort tests by size before slicing them, we skip the _smallest_ test.

Reframe the problem to instead talk about "slice number" and "total
number of slices". For all of our use cases this is semantically
equivalent, but it allows us to perform some verifications:

  - The total number of slices must be greater than 1.

  - The selected slice must be between 1 &lt;= nr &lt;= slices_total.

As the indices are now one-based it means that GitLab's CI is fixed.
The GitHub workflow is updated accordingly.

Helped-by: Johannes Schindelin &lt;johannes.schindelin@gmx.de&gt;
Signed-off-by: Patrick Steinhardt &lt;ps@pks.im&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
<entry>
<title>Merge branch 'am/doc-github-contributiong-link-to-submittingpatches'</title>
<updated>2026-02-11T20:29:07Z</updated>
<author>
<name>Junio C Hamano</name>
<email>gitster@pobox.com</email>
</author>
<published>2026-02-11T20:29:07Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=18df0fa3ca89be67db76d13fd313761e79f6bb11'/>
<id>urn:sha1:18df0fa3ca89be67db76d13fd313761e79f6bb11</id>
<content type='text'>
GitHub repository banner update.

* am/doc-github-contributiong-link-to-submittingpatches:
  .github/CONTRIBUTING.md: link to SubmittingPatches on git-scm.com
</content>
</entry>
<entry>
<title>.github/CONTRIBUTING.md: link to SubmittingPatches on git-scm.com</title>
<updated>2026-02-02T17:56:44Z</updated>
<author>
<name>Abdalrhman Mohamed</name>
<email>eng.abdalrhman.abdalmonem@gmail.com</email>
</author>
<published>2026-02-01T00:26:43Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=2d45507f15866f5a0755f5983d214116c7b2c05f'/>
<id>urn:sha1:2d45507f15866f5a0755f5983d214116c7b2c05f</id>
<content type='text'>
The relative link to SubmittingPatches is broken when viewed through
GitHub's specialized "Contributing" tab. Update the link to point to
the documentation on git-scm.com to be consistent with other links in
the same file. Also, wrap the line to improve readability.

Signed-off-by: Abdalrhman Mohamed &lt;Eng.Abdalrhman.Abdalmonem@gmail.com&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
<entry>
<title>Merge branch 'js/ci-show-breakage-in-dockerized-jobs'</title>
<updated>2025-12-05T05:49:58Z</updated>
<author>
<name>Junio C Hamano</name>
<email>gitster@pobox.com</email>
</author>
<published>2025-12-05T05:49:58Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=85f99338e107a36650257787a350821acc36a81d'/>
<id>urn:sha1:85f99338e107a36650257787a350821acc36a81d</id>
<content type='text'>
Dockerised jobs at the GitHub Actions CI have been taught to show
more details of failed tests.

* js/ci-show-breakage-in-dockerized-jobs:
  ci(dockerized): do show the result of failing tests again
</content>
</entry>
</feed>
