Skip to content

Update version #60

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
BNAndras wants to merge 10 commits into from
Closed

Update version #60

Changes from all commits
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
ac81b32
Bump pyret-npm version
BNAndras Jul 20, 2025
3577923
Bunp alpine version
BNAndras Jul 20, 2025
b05d918
try node image
BNAndras Jul 20, 2025
0a7e776
remove apk installs
BNAndras Jul 20, 2025
5068a95
Add additional dependencies
BNAndras Jul 20, 2025
e26929a
Add dependencies
BNAndras Jul 20, 2025
d77f1bd
add cairo dev
BNAndras Jul 20, 2025
8a4b6b1
Add pango
BNAndras Jul 20, 2025
836c08e
Try pango dev
BNAndras Jul 20, 2025
88cf0ba
retrigger CI
BNAndras Jul 20, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
16 changes: 13 additions & 3 deletions Dockerfile
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,8 +1,18 @@
FROM alpine:3.17
ARG PYRET_NPM_VERSION=0.0.27
FROM node:lts-alpine3.22
ARG PYRET_NPM_VERSION=0.0.50

# install packages required to run the tests
RUN apk add --no-cache jq coreutils nodejs npm && \
RUN apk add --no-cache \
cairo-dev \
coreutils \
g++ \
jq \
pango-dev \
pkgconfig \
pixman-dev \
python3 \
make \
g++ && \
Comment on lines +5 to +15
Copy link
Copy Markdown

@glennj glennj Jul 20, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

g++ twice. Not a problem, but untidy.

Copy link
Copy Markdown
Member Author

@BNAndras BNAndras Jul 20, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That’s fair, but I’m still heavily reworking this. We’re pulling in a lot of new dependencies unfortunately because the image library for CPO was merged into the standard library.

npm install -g pyret-npm@$PYRET_NPM_VERSION

WORKDIR /opt/test-runner
Expand Down
Loading