Skip to content

Commit

Permalink
Take ownership of the repo in the container
Browse files Browse the repository at this point in the history
And add a fixme about how the test suite could be improved to not
require this.
  • Loading branch information
EliahKagan committed Nov 17, 2024
1 parent eaf2243 commit 47f3819
Showing 1 changed file with 6 additions and 0 deletions.
6 changes: 6 additions & 0 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -219,6 +219,12 @@ jobs:
apt-get install --no-install-recommends -y -- "${prerequisites[@]}"
shell: bash
- uses: actions/checkout@v4
- name: Take ownership of repo
# FIXME: Eliminate the need for this by improving the test suite not to rely on ownership
# of preexisting files, nor on whether, where, or how we are contained in a repo (except
# where gix-testtools checks ignores to suppress archives). Note that `safe.directory`
# isn't a fix, due to config and environment sanitization when running fixture scripts.
run: chown -R "$(whoami)" -- "$GITHUB_WORKSPACE"
- uses: dtolnay/rust-toolchain@stable
with:
toolchain: stable-i686-unknown-linux-gnu # Otherwise it may misdetect based on the amd64 kernel.
Expand Down

0 comments on commit 47f3819

Please sign in to comment.