Skip to content
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.

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

Update to 5.4.1 #320

Merged
merged 2 commits into from
Jul 20, 2022
Merged

Update to 5.4.1 #320

merged 2 commits into from
Jul 20, 2022

Conversation

jmaitrehenry
Copy link
Contributor

No description provided.

@tianon
Copy link
Member

tianon commented Jul 20, 2022

Thanks! I dug a little into the CI failure, and found it's currently failing thanks to TryGhost/node-sqlite3#1625 -- it appears that it works if we update to Alpine 3.16, so I'm about to push a new commit to your branch to try that. 🙈

@tianon tianon merged commit 434140b into docker-library:master Jul 20, 2022
@tianon
Copy link
Member

tianon commented Jul 20, 2022

Success!! Thanks again for the contribution 👍 💪

docker-library-bot added a commit to docker-library-bot/official-images that referenced this pull request Jul 20, 2022
Changes:

- docker-library/ghost@434140b: Merge pull request docker-library/ghost#320 from jmaitrehenry/master
- docker-library/ghost@3c0190d: Update to Alpine 3.16
- docker-library/ghost@f5591a1: Update to 5.4.1, ghost-cli 1.21.0
@jmaitrehenry
Copy link
Contributor Author

jmaitrehenry commented Jul 21, 2022

@tianon I tested the debian build only as I use it for my personal blog as a base image, but I didn't test the alpine image. I assume it works out of the box, next time, I will run it too to be sure it works.

Thanks for the fix and merge!

@tianon
Copy link
Member

tianon commented Jul 21, 2022

It's OK! That's what CI is for. 😄

Typically, these updates are picked up automatically by @docker-library-bot, but in this case there was a problem with the build so it (correctly) didn't auto-apply the commit. 😅

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.

2 participants