Skip to content

Commit cac5e8a

Browse files
wingologanek
authored andcommitted
Fix compile-tests workflow
Moving source files out of the built dir effectively removed the built dir, causing merge conflicts in the workflow that rebuilt the wasm tests which were supposed to be in that dir. Fix by doing a `--strategy ours` merge, then forcibly resetting the tree state to the main branch, then checking out the built files. This ensures that the tree contains precisely the upstream source code, plus our built files, while keeping the parent tree.
1 parent 39773f0 commit cac5e8a

File tree

1 file changed

+7
-1
lines changed

1 file changed

+7
-1
lines changed

.github/workflows/compile-tests.yml

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -140,7 +140,13 @@ jobs:
140140
uses: ./.github/actions/git-config
141141

142142
- name: Merge main branch changes
143-
run: git merge ${{ github.sha }}
143+
run: |
144+
git merge --no-commit -Sours ${{ github.sha }}
145+
git checkout ${{ github.sha }} .
146+
git checkout HEAD tests/assemblyscript/testsuite
147+
git checkout HEAD tests/c/testsuite
148+
git checkout HEAD tests/rust/testsuite
149+
git commit -m "Merge commit ${{ github.sha }} into prod/testsuite-base"
144150
145151
- name: Remove existing binaries
146152
run: rm -rf tests/${{ matrix.suite }}/testsuite

0 commit comments

Comments
 (0)