Skip to content

Commit 367b07d

Browse files
committed
CI: remove old git workaround
Apparently it's only needed in the SanityCheck job (because of commitlint).
1 parent c613b26 commit 367b07d

File tree

1 file changed

+5
-7
lines changed

1 file changed

+5
-7
lines changed

.github/workflows/CI.yml

Lines changed: 5 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -22,9 +22,6 @@ jobs:
2222
apt update
2323
apt install --yes sudo
2424
sudo apt install --yes --no-install-recommends git
25-
# workaround for https://github.com/actions/runner/issues/2033
26-
- name: ownership workaround
27-
run: git config --global --add safe.directory '*'
2825
2926
- name: Setup .NET
3027
run: |
@@ -161,9 +158,13 @@ jobs:
161158
- name: Install required dependencies
162159
run: |
163160
apt update && apt install --yes sudo
164-
sudo apt install --yes --no-install-recommends git
165161
166162
sudo apt install --yes --no-install-recommends npm curl
163+
164+
sudo apt install --yes --no-install-recommends git
165+
# workaround for https://github.com/actions/runner/issues/2033
166+
git config --global --add safe.directory '*'
167+
167168
# need to update nodejs because with ubuntu's default nodejs version we would get this error:
168169
# ```
169170
# + npx commitlint --from HEAD~1 --to HEAD --verbose
@@ -182,9 +183,6 @@ jobs:
182183
submodules: recursive
183184
# needed because of commit-lint, see https://github.com/conventional-changelog/commitlint/issues/3376
184185
fetch-depth: 0
185-
# workaround for https://github.com/actions/runner/issues/2033
186-
- name: ownership workaround
187-
run: git config --global --add safe.directory '*'
188186
- name: Print versions
189187
run: |
190188
git --version

0 commit comments

Comments
 (0)