Skip to content

Commit 316fbcc

Browse files
authored
Merge pull request #50 from cryptosense/fix-version-subst
Fix version substitution on Windows
2 parents 670a99d + 29d10a3 commit 316fbcc

File tree

1 file changed

+6
-2
lines changed

1 file changed

+6
-2
lines changed

.github/workflows/main.yml

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -34,10 +34,14 @@ jobs:
3434
- name: Install the dependencies
3535
run: opam install --deps-only --with-test --locked --lock-suffix win.locked cs_api_client
3636
- name: Insert the version number
37-
run: ./ci/subst.bash "${{ steps.build_version.outputs.value }}"
38-
if: startsWith(github.ref, 'refs/tags/')
37+
shell: bash
38+
env:
39+
SHELLOPTS: igncr
40+
run: bash ci/subst.bash "${{ steps.build_version.outputs.value }}"
3941
- name: Build the executable and run the tests
4042
run: opam exec -- dune build @all @runtest
43+
- name: Run --version
44+
run: _build/install/default/bin/cs-api.exe --version
4145
- name: Copy the artifacts to a directory (in powershell)
4246
run: |
4347
New-Item -ItemType Directory -ErrorAction SilentlyContinue build/cs-api-${{ steps.build_version.outputs.value }}-windows

0 commit comments

Comments
 (0)