Skip to content

Commit 59b7cbe

Browse files
committed
tidy up dub.yml
1 parent 39b86f5 commit 59b7cbe

File tree

1 file changed

+2
-24
lines changed

1 file changed

+2
-24
lines changed

.github/workflows/dub.yml

Lines changed: 2 additions & 24 deletions
Original file line numberDiff line numberDiff line change
@@ -35,31 +35,12 @@ jobs:
3535
with:
3636
compiler: ${{ matrix.compiler }}
3737

38-
# - name: Install dependencies on Ubuntu
39-
# if: startsWith(matrix.os, 'ubuntu')
40-
# run: sudo apt-get update && sudo apt-get install libevent-dev -y
41-
42-
# - name: Install dependencies on Mac OSX
43-
# if: startsWith(matrix.os, 'macos')
44-
# run: brew bundle
45-
4638
- name: Upgrade dub dependencies
4739
uses: WebFreak001/[email protected]
4840

4941
- name: Build Library
5042
run: dub build --build=release --config=library
5143

52-
## Turns out the unittest-vibe-ut tried to connect to an actualy MySQL on 172.18.0.1 so it's not
53-
## actually a unit test at all. It's an integration test and should be pulled out of the codebase.
54-
# - name: Run unittest-vibe-ut
55-
# run: dub run -c unittest-vibe-ut -- -t
56-
57-
# - name: Run Tests (phobos)
58-
# run: ./run_tests --mode=phobos
59-
60-
# - name: Run Tests (vibe)
61-
# run: ./run_tests --mode=vibe
62-
6344
# cache
6445
- uses: WebFreak001/[email protected]
6546
if: startsWith(matrix.os, 'windows')
@@ -102,10 +83,6 @@ jobs:
10283
with:
10384
compiler: ${{ matrix.compiler }}
10485

105-
# - name: Install dependencies on Ubuntu
106-
# if: startsWith(matrix.os, 'ubuntu')
107-
# run: sudo apt-get update && sudo apt-get install libevent-dev -y
108-
10986
- name: Upgrade dub dependencies
11087
uses: WebFreak001/[email protected]
11188

@@ -175,7 +152,8 @@ jobs:
175152
run: sudo apt-get update && sudo apt-get install libevent-dev -y
176153

177154
## Turns out the unittest-vibe-ut tried to connect to an actualy MySQL on 172.18.0.1 so it's not
178-
## actually a unit test at all. It's an integration test and should be pulled out of the codebase.
155+
## actually a unit test at all. It's an integration test and should be pulled out from the main
156+
## codebase into a separate sub module
179157
- name: Run unittest-vibe-ut
180158
env:
181159
MYSQL_PORT: ${{ job.services.mysql.ports[3306] }}

0 commit comments

Comments
 (0)