Skip to content

Update config for Alpine 3.9#988

Merged
PeterDaveHello merged 1 commit intonodejs:masterfrom
chorrell:alpine-3.9
Jan 31, 2019
Merged

Update config for Alpine 3.9#988
PeterDaveHello merged 1 commit intonodejs:masterfrom
chorrell:alpine-3.9

Conversation

@chorrell
Copy link
Copy Markdown
Contributor

Update for Alpine 3.9:

Closes #987

Do we want a DockerHub PR for this or should we include it with the next Node.js updates? I think we decided that we'd err on the side of keeping Alpine current.

@chorrell chorrell requested a review from a team January 31, 2019 15:01
@LaurentGoderre
Copy link
Copy Markdown
Member

I would vote for next version personally.

Copy link
Copy Markdown
Member

@PeterDaveHello PeterDaveHello left a comment

Choose a reason for hiding this comment

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

Maybe later?

@chorrell
Copy link
Copy Markdown
Contributor Author

Should I just update the config file and drop the Dockerfile changes?

@PeterDaveHello
Copy link
Copy Markdown
Member

Sounds better 👍

@LaurentGoderre
Copy link
Copy Markdown
Member

@chorrell you can just update this PR instead of opening a new one

@chorrell
Copy link
Copy Markdown
Contributor Author

I updated my PR to just update the config file

@PeterDaveHello PeterDaveHello changed the title Alpine 3.9 Update config for Alpine 3.9 Jan 31, 2019
@PeterDaveHello PeterDaveHello merged commit d4d52ac into nodejs:master Jan 31, 2019
@lbourdages
Copy link
Copy Markdown

Not sure if I am missing something, but the dockerfile still uses 3.8 🤔 https://github.com/nodejs/docker-node/blob/master/11/alpine/Dockerfile

@PeterDaveHello
Copy link
Copy Markdown
Member

New version not released yet.

@IdanAdar
Copy link
Copy Markdown

IdanAdar commented Feb 17, 2019

Do we know when will it be officially released? Looking for node:8-alpine with alpine 3.9.

@LaurentGoderre
Copy link
Copy Markdown
Member

LaurentGoderre commented Feb 17, 2019

@MylesBorins do you know if a version 8 update is planned soon?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants