Skip to content

Commit 47f3819

Browse files
committed
Take ownership of the repo in the container
And add a fixme about how the test suite could be improved to not require this.
1 parent eaf2243 commit 47f3819

File tree

1 file changed

+6
-0
lines changed

1 file changed

+6
-0
lines changed

.github/workflows/ci.yml

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -219,6 +219,12 @@ jobs:
219219
apt-get install --no-install-recommends -y -- "${prerequisites[@]}"
220220
shell: bash
221221
- uses: actions/checkout@v4
222+
- name: Take ownership of repo
223+
# FIXME: Eliminate the need for this by improving the test suite not to rely on ownership
224+
# of preexisting files, nor on whether, where, or how we are contained in a repo (except
225+
# where gix-testtools checks ignores to suppress archives). Note that `safe.directory`
226+
# isn't a fix, due to config and environment sanitization when running fixture scripts.
227+
run: chown -R "$(whoami)" -- "$GITHUB_WORKSPACE"
222228
- uses: dtolnay/rust-toolchain@stable
223229
with:
224230
toolchain: stable-i686-unknown-linux-gnu # Otherwise it may misdetect based on the amd64 kernel.

0 commit comments

Comments
 (0)