Skip to content

Add Codecov coverage upload to CI#235

Merged
dhalperi merged 1 commit into
masterfrom
spr/master/aa7721ed
Mar 26, 2026
Merged

Add Codecov coverage upload to CI#235
dhalperi merged 1 commit into
masterfrom
spr/master/aa7721ed

Conversation

@dhalperi

@dhalperi dhalperi commented Mar 23, 2026

Copy link
Copy Markdown
Member

Generate XML coverage report and upload to Codecov from the
Python 3.13 matrix job. Uses the Codecov GitHub App for auth
(no token needed).


Prompt:

Can we add code coverage to this github repository? I think it
used to be supported, but stopped. I just reinstalled the github
app to the Intentionet organization, but I don't have any tokens
or whatever yet.

This change is Reviewable

@dhalperi dhalperi force-pushed the spr/master/aa7721ed branch 2 times, most recently from 7d6b51a to 526a522 Compare March 23, 2026 19:13

@dhalperi dhalperi left a comment

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

@dhalperi reviewed 2 files and all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved.

Generate XML coverage report and upload to Codecov from the
Python 3.13 matrix job. Uses the Codecov GitHub App for auth
(no token needed).

----

Prompt:
```
Can we add code coverage to this github repository? I think it
used to be supported, but stopped. I just reinstalled the github
app to the Intentionet organization, but I don't have any tokens
or whatever yet.
```

commit-id:aa7721ed
@dhalperi dhalperi force-pushed the spr/master/aa7721ed branch from 526a522 to 1323906 Compare March 23, 2026 19:15
@codecov

codecov Bot commented Mar 26, 2026

Copy link
Copy Markdown

Welcome to Codecov 🎉

Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.

Thanks for integrating Codecov - We've got you covered ☂️

@dhalperi dhalperi closed this Mar 26, 2026
@dhalperi dhalperi reopened this Mar 26, 2026
@dhalperi

Copy link
Copy Markdown
Member Author

Ok I'll trust the robot and see if it works once merged.

@dhalperi dhalperi left a comment

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

@dhalperi reviewed 1 file and all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved.

@dhalperi dhalperi enabled auto-merge (squash) March 26, 2026 23:26
@dhalperi dhalperi disabled auto-merge March 26, 2026 23:26
@dhalperi dhalperi merged commit 30be448 into master Mar 26, 2026
32 of 36 checks passed
@dhalperi dhalperi deleted the spr/master/aa7721ed branch March 26, 2026 23:27
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