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

docker: fix run.sh script #1

Merged
merged 5 commits into from
Feb 28, 2025
Merged

docker: fix run.sh script #1

merged 5 commits into from
Feb 28, 2025

Conversation

hansbinderup
Copy link
Owner

Point at the correct container.
Also, let's see if the CI works now :)

Point at the correct container.
Also, let's see if the CI works now :)

Signed-off-by: Hans Binderup <[email protected]>
@hansbinderup hansbinderup force-pushed the hansbinderup/test-pr branch 4 times, most recently from 1804e87 to c8a71a6 Compare February 28, 2025 14:26
1. We've added a user, so we should probably use it.
2. Problem seems to come from checkout action - maybe this works?

Signed-off-by: Hans Binderup <[email protected]>
Was previously a submodule but we should use wrap packages instead.

Signed-off-by: Hans Binderup <[email protected]>
…actions

Safety feature from git. Let's handle that in the container.

Signed-off-by: Hans Binderup <[email protected]>
For CI to pick up build errors etc. it's necessary to make the scripts
fail as errors.

Signed-off-by: Hans Binderup <[email protected]>
@hansbinderup hansbinderup force-pushed the hansbinderup/test-pr branch 2 times, most recently from a8c3d44 to e66a8b2 Compare February 28, 2025 15:30
@hansbinderup hansbinderup merged commit 9b202aa into main Feb 28, 2025
4 of 5 checks passed
@hansbinderup hansbinderup deleted the hansbinderup/test-pr branch February 28, 2025 15:35
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.

1 participant