Skip to content

fixed issue 724#1333

Draft
rohan-devbuilds wants to merge 1 commit into
collective:mainfrom
rohan-devbuilds:main
Draft

fixed issue 724#1333
rohan-devbuilds wants to merge 1 commit into
collective:mainfrom
rohan-devbuilds:main

Conversation

@rohan-devbuilds

@rohan-devbuilds rohan-devbuilds commented Apr 22, 2026

Copy link
Copy Markdown

.

Description

Coverage generation is fine — the XML and HTML reports both get created successfully. The failure is in the Coveralls upload step. It tries to download a pre-built coveralls-linux.tar.gz binary at runtime and that download is silently failing, so by the time sha256sum runs the file doesn't exist.
The easiest fix is to replace the action with a direct pip install instead:

yaml- name: Upload coverage to Coveralls
  env:
    GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
  run: |
    pip install coveralls
    coveralls

GITHUB_TOKEN is available by default in Actions so no extra setup needed. This skips the binary download entirely and is more reliable for Python projects.

Checklist

  • I've added a change log entry to CHANGES.rst.
  • I've added or updated tests if applicable.
  • I've run and ensured all tests pass locally by following Run tests.
  • I've added or edited documentation, both as docstrings to be rendered in the API documentation and narrative documentation, as necessary.

Additional information

Upload screenshots, videos, links to documentation, or any other relevant information.

@read-the-docs-community

Copy link
Copy Markdown

Documentation build overview

📚 icalendar | 🛠️ Build #32375599 | 📁 Comparing 0b354dc against latest (f77657c)

  🔍 Preview build  

5 files changed · ± 5 modified

± Modified

@niccokunzmann niccokunzmann marked this pull request as draft April 22, 2026 20:02
@niccokunzmann

Copy link
Copy Markdown
Member

@rohan-devbuilds Thanks for the PR. I don't really understand the change made here. The test should run. Could you remove all that text from the PR description and do what is needed.

@niccokunzmann niccokunzmann added the ai-suspicion This contribution is possibly created with lots of AI help without enough human understanding. label Apr 22, 2026
@niccokunzmann

Copy link
Copy Markdown
Member

Hi, we created a new release, please move your edits in CHANGES.rst to the new section.

1 similar comment
@niccokunzmann

Copy link
Copy Markdown
Member

Hi, we created a new release, please move your edits in CHANGES.rst to the new section.

@stevepiercy

Copy link
Copy Markdown
Member

@rohan-devbuilds would you please respond to @niccokunzmann's comments? Do you need assistance to proceed? Please let us know. Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ai-suspicion This contribution is possibly created with lots of AI help without enough human understanding.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG] CI: Tests fail because coverage cannot be submitted

3 participants