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

Use Debian Slim as base image #148

Closed
wants to merge 10 commits into from
Closed

Use Debian Slim as base image #148

wants to merge 10 commits into from

Conversation

kroese
Copy link
Contributor

@kroese kroese commented Oct 27, 2023

Should make the docker image a lot smaller

@bokysan
Copy link
Owner

bokysan commented Oct 27, 2023

Thank you.

Could I kindly ask you to rebase and squash?

@kroese
Copy link
Contributor Author

kroese commented Oct 27, 2023

I rebased it now, but I am making these commits through the Github Web UI so I dont think I can squash them

Dockerfile Outdated
@@ -1,6 +1,6 @@
# syntax=docker/dockerfile:1.2

ARG BASE_IMAGE=debian:latest
ARG BASE_IMAGE=debian-slim:latest
Copy link
Owner

Choose a reason for hiding this comment

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

This is wrong. That's why the integration test is failing.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed now

@bokysan
Copy link
Owner

bokysan commented Oct 27, 2023

Pull request seems to work, but we really need to rebase and squash it.

There are literary 5 changes but 10 commits and I cannot merge it because it says "This branch cannot be rebased due to conflicts".

@kroese
Copy link
Contributor Author

kroese commented Oct 27, 2023

Replaced by #149

@kroese kroese closed this Oct 27, 2023
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