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

cppSlint and rustSlint production Dockerfile fixes + Github Actions Checks #108

Merged
merged 8 commits into from
Sep 4, 2023

Conversation

microhobby
Copy link
Collaborator

@microhobby microhobby commented Sep 1, 2023

fixes #95
fixes #101
fixes #97

@microhobby microhobby added the bug Something isn't working label Sep 1, 2023
@microhobby microhobby self-assigned this Sep 1, 2023
@microhobby microhobby added this to the v2.2.0 milestone Sep 1, 2023
@microhobby microhobby marked this pull request as draft September 1, 2023 03:10
@microhobby
Copy link
Collaborator Author

I'm setting this as draft because I think that is time to take advantage to implement some Github Actions tests to not have this fails again.

Copy link
Contributor

@tronical tronical left a comment

Choose a reason for hiding this comment

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

Thank you for the fixes! We’re planning a release next Monday so the timing of this is excellent;)

rustSlint/Dockerfile Outdated Show resolved Hide resolved
For the production image we need to use the default
torizon/wayland-base image as base, because we do not need
any other development tool, we are only shipping the
slint binary.

Signed-off-by: Matheus Castello <[email protected]>
@microhobby microhobby force-pushed the castello/slint branch 9 times, most recently from 897dfaf to f750f7f Compare September 2, 2023 01:24
@microhobby microhobby changed the title cppSlint and rustSlint production Dockerfile fixes cppSlint and rustSlint production Dockerfile fixes + Github Actions Checks Sep 2, 2023
For the production image we need to use the default
torizon/wayland-base mage as base, because we do not need
any other development tool, we are only shipping the
application.

Signed-off-by: Matheus Castello <[email protected]>
Signed-off-by: Matheus Castello <[email protected]>
@microhobby
Copy link
Collaborator Author

Hey @tronical I bumped the Slint version for the Dockerfiles in the 0af1845

Let me know if I need to bump it into some other template file.

@microhobby microhobby marked this pull request as ready for review September 4, 2023 20:02
@microhobby microhobby requested review from tronical and removed request for tronical and andreriesco September 4, 2023 20:02
@microhobby microhobby merged commit 1c4e9b6 into dev Sep 4, 2023
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants