Skip to content

Commit ecc9e39

Browse files
committed
Update docker file to include luasocket instead of installing via script
1 parent 84164aa commit ecc9e39

2 files changed

Lines changed: 3 additions & 7 deletions

File tree

.github/workflows/test.yml

Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -12,12 +12,7 @@ jobs:
1212
steps:
1313
- name: Checkout
1414
uses: actions/checkout@v4
15-
- name: Install LuaSocket
16-
run: |
17-
apt-get update
18-
apt-get install -y luarocks
19-
luarocks install luasocket
2015
- name: Run tests
2116
run: busted --lua=luajit
2217
- name: Report coverage
23-
run: cd src; luacov-coveralls --repo-token=${{ secrets.github_token }} -e TestData -e Data -e runtime
18+
run: cd src; luacov-coveralls --repo-token=${{ secrets.github_token }} -e TestData -e Data -e runtime -e runtime

Dockerfile

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,8 @@ RUN --mount=type=cache,from=luarocks,source=/opt,target=/opt make -C /opt/luaroc
3232
RUN luarocks install busted 2.2.0-1;\
3333
luarocks install cluacov 0.1.2-1;\
3434
luarocks install luacov-coveralls 0.2.3-1;\
35-
luarocks install luautf8 0.1.6-1
35+
luarocks install luautf8 0.1.6-1;\
36+
luarocks install luasocket
3637

3738
RUN --mount=type=cache,from=emmyluadebugger,source=/opt,target=/opt make -C /opt/EmmyLuaDebugger/build/ install
3839
RUN --mount=type=cache,from=luajit,source=/opt,target=/opt make -C /opt/LuaJIT/ install

0 commit comments

Comments
 (0)