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

(feat): add ubuntu22 image + build process, update docs #85

Merged
merged 4 commits into from
Dec 12, 2023

Conversation

traktuner
Copy link
Collaborator

Included in this PR:
-) new :ubuntu22 docker tag with jlesage's ubuntu22 baseimage-gui
-) updated build process
-) readme + changelog updated

The ubuntu20 image will still serve as the default image, as discussed.
The ubuntu18 and ubuntu22 images can only be used with the corresponding tags.

@JonathanTreffler
Copy link
Owner

JonathanTreffler commented Dec 12, 2023

Thank you for the PR !

Except for the requested change (see review) this looks great 👍

Copy link
Owner

@JonathanTreffler JonathanTreffler left a comment

Choose a reason for hiding this comment

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

The Build Action for the ubuntu 22 action currently fails for this PR, because it tries to push to the tag :pr-85:ubuntu22. Please disabled PR image tags for the ubuntu 22 image, so it behaves like ubuntu18, which just builds the image in PRs, but doesn't try to push it.

@traktuner traktuner force-pushed the feat/add-ubuntu22-dockerfile branch from 216f552 to 6a0e9bc Compare December 12, 2023 14:30
@traktuner
Copy link
Collaborator Author

Also fixes #71
Bildschirmfoto 2023-12-12 um 16 00 30

Copy link
Owner

@JonathanTreffler JonathanTreffler left a comment

Choose a reason for hiding this comment

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

👍

@JonathanTreffler JonathanTreffler merged commit d0f1c7f into main Dec 12, 2023
4 checks passed
@traktuner traktuner deleted the feat/add-ubuntu22-dockerfile branch December 13, 2023 06:49
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