Skip to content

Adds additional support for Github enterprise usecases #548

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

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

Conversation

daniel-anya
Copy link
Contributor

@daniel-anya daniel-anya commented May 15, 2025

5764cba: Adds support for figuring out a Github host from the GITHUB_BASE_URL config, and using this value in generated coverage artifacts. The usecase here is that, when this action is used in the enterprise environment, coverage artifacts like html files and README docs point to github.com/... which is not what we'd want.

07a88e1: Adds support for saving coverage artifacts on PR merged to default branch events which is functionally equivalent to the currently supported path of saving coverage artifacts on commit pushed to default branch events.

78a7324: Adds support for generating Github Pages based coverage report links.

Tested all changes in a Github enterprise environment and they've been working fine for a while now.

daniel-anya and others added 4 commits May 15, 2025 11:51
…st merge to the default branch.

This is functionally equivalent to the currently supported pattern of pushing to the default branch.
Adds support for figuring out a github host from the `GITHUB_BASE_URL` config and using this value in generated coverage artifacts rather than hardcoding them to "github.com"
Copy link

End-to-end public repo

Admin commands cheatsheet:

  • /e2e (in approved PR review body): Trigger end-to-end tests on external contributions
  • /invite (in comment): Invite the author & admins to the end-to-end private repo

Copy link

github-actions bot commented May 15, 2025

Coverage report

Click to see where and how coverage changed

FileStatementsMissingCoverageCoverage
(new stmts)
Lines missing
  coverage_comment
  activity.py
  github.py
  main.py 77-81
  settings.py
  storage.py
  template.py
Project Total  

This report was generated by python-coverage-comment-action

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