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

Test new docker workflow #46

Open
wants to merge 6 commits into
base: main
Choose a base branch
from
Open

Test new docker workflow #46

wants to merge 6 commits into from

Conversation

Bisaloo
Copy link
Contributor

@Bisaloo Bisaloo commented Jul 25, 2023

No description provided.

@Bisaloo Bisaloo closed this Jul 25, 2023
@Bisaloo Bisaloo reopened this Jul 25, 2023
@codecov
Copy link

codecov bot commented Jul 26, 2023

Codecov Report

Merging #46 (32afe32) into main (3a2609d) will not change coverage.
Report is 3 commits behind head on main.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #46   +/-   ##
=======================================
  Coverage   97.29%   97.29%           
=======================================
  Files          17       17           
  Lines        1701     1701           
=======================================
  Hits         1655     1655           
  Misses         46       46           

@@ -14,6 +14,9 @@ jobs:
container: ghcr.io/bisaloo/libbi-docker:main
name: docker
env:
# Avoid interaction requests from apt. It's better to set it here than in
# the Docker img: https://github.com/moby/moby/issues/4032
DEBIAN_FRONTEND: noninteractive
Copy link
Owner

Choose a reason for hiding this comment

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

Might be a problem that it's not passed to the sudo env in https://github.com/sbfnk/rbi/actions/runs/6081518618/job/16497330229#step:5:1481?

A (not hugely satisfactory) solution might be to install tzdata in the docker image instead.

@Bisaloo Bisaloo closed this Sep 6, 2023
@Bisaloo Bisaloo reopened this Sep 6, 2023
@Bisaloo Bisaloo closed this Sep 6, 2023
@Bisaloo Bisaloo reopened this Sep 6, 2023
@Bisaloo Bisaloo marked this pull request as ready for review September 12, 2023 12:41
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.

None yet

2 participants