Skip to content

Commit

Permalink
Add git safe directory to workaround github actions ownership issues (#…
Browse files Browse the repository at this point in the history
…26)

* Add git safe directory to workaround github actions ownership issues

* Update CI action versions

* Update tests to reflect coveralls-python no longer retrying on 422 responses
  • Loading branch information
djhoese committed May 9, 2024
1 parent 8799c9f commit 65c1672
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 4 deletions.
6 changes: 3 additions & 3 deletions .github/workflows/push.yml
Original file line number Diff line number Diff line change
Expand Up @@ -5,8 +5,8 @@ jobs:
test:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v2
- uses: actions/setup-python@v2
- uses: actions/checkout@v3
- uses: actions/setup-python@v3

- name: Unit tests
run: make test PYTHON_WITH_VERSION=python3
Expand Down Expand Up @@ -38,7 +38,7 @@ jobs:
needs: test
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v2
- uses: actions/checkout@v3
- name: Coveralls Finished
uses: ./
with:
Expand Down
2 changes: 2 additions & 0 deletions Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -5,4 +5,6 @@ RUN python3 -m pip install Cython

RUN python3 -m pip install "coverage[toml]"

RUN git config --system --add safe.directory "/github/workspace"

ENTRYPOINT ["/src/entrypoint.py"]
2 changes: 1 addition & 1 deletion tests/test_entrypoint.py
Original file line number Diff line number Diff line change
Expand Up @@ -253,7 +253,7 @@ def test_status_code_422(self):
), patch_log() as m_log:
entrypoint.run_coveralls(repo_token="TOKEN")
# coveralls package will retry once per service we call it with
assert m_post.call_count == 4
assert m_post.call_count == 2
assert m_log.error.call_args_list == [
mock.call("Failed to submit coverage", exc_info=None)
]
Expand Down

0 comments on commit 65c1672

Please sign in to comment.