Skip to content

Conversation

@chorrell
Copy link
Contributor

Includes Yarn update for 1.7.0

Includes Yarn update for 1.7.0
@chorrell chorrell requested a review from a team May 24, 2018 00:01
LaurentGoderre

This comment was marked as off-topic.

@chorrell
Copy link
Contributor Author

@chorrell chorrell merged commit 9e2a4c9 into master May 24, 2018
@chorrell chorrell deleted the node-10.2.0-yarn-1.7.0 branch May 24, 2018 01:20
@chorrell
Copy link
Contributor Author

Hmm, problem with the deploy stage:

https://travis-ci.org/nodejs/docker-node/jobs/382969501

$ ./generate-stackbrew-pr.sh
Cloning...
./generate-stackbrew-pr.sh: line 121: {library/node}: No such file or directory

@LaurentGoderre ☝️

I'll do the docker hub PR if you can look into that above

@chorrell
Copy link
Contributor Author

Ah, I see the issue:

https://github.com/nodejs/docker-node/blob/master/generate-stackbrew-pr.sh#L121

This:

echo "${stackbrew}" >"{$IMAGES_FILE}"

Should be this

echo "${stackbrew}" >"${IMAGES_FILE}"

@chorrell
Copy link
Contributor Author

Docker hub PR: docker-library/official-images#4388

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants