Skip to content

Update code cov, include ruff#91

Merged
catanzaromj merged 16 commits into
scikit-tda:masterfrom
catanzaromj:update_code_cov
Jan 20, 2026
Merged

Update code cov, include ruff#91
catanzaromj merged 16 commits into
scikit-tda:masterfrom
catanzaromj:update_code_cov

Conversation

@catanzaromj
Copy link
Copy Markdown
Contributor

@catanzaromj catanzaromj commented Oct 1, 2025

This PR fixes a bug with the old codecov github action for uploading code coverage. It also introduces ruff into persim for both linting and formatting. Both of these are modernizations to get persim up to date with common best practices.

@codecov
Copy link
Copy Markdown

codecov Bot commented Oct 1, 2025 •
edited
Loading

Codecov Report

❌ Patch coverage is 81.55340% with 19 lines in your changes missing coverage. Please review.
✅ Project coverage is 79.40%. Comparing base (39ec0bf) to head (b914492).
⚠️ Report is 17 commits behind head on master.

Files with missing lines Patch % Lines
persim/images_kernels.py 50.00% 8 Missing ⚠️
persim/gromov_hausdorff.py 77.77% 3 Missing and 1 partial ⚠️
persim/persistent_entropy.py 66.66% 1 Missing and 3 partials ⚠️
persim/visuals.py 83.33% 1 Missing and 1 partial ⚠️
persim/landscapes/base.py 0.00% 1 Missing ⚠️
Additional details and impacted files 
@@            Coverage Diff             @@
##           master      #91      +/-   ##
==========================================
- Coverage   79.40%   79.40%   -0.01%     
==========================================
  Files          20       20              
  Lines        1486     1481       -5     
  Branches      327      271      -56     
==========================================
- Hits         1180     1176       -4     
- Misses        228      229       +1     
+ Partials       78       76       -2

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@catanzaromj catanzaromj marked this pull request as ready for review October 1, 2025 22:50
@catanzaromj catanzaromj merged commit 2d50e66 into scikit-tda:master Jan 20, 2026
12 checks passed
catanzaromj pushed a commit that referenced this pull request Mar 7, 2026
@dependabot @catanzaromj
ci: Bump actions/download-artifact from 5 to 6
ea72900 
Bumps [actions/download-artifact](https://github.com/actions/download-artifact) from 5 to 6.
- [Release notes](https://github.com/actions/download-artifact/releases)
- [Commits](actions/download-artifact@v5...v6)

---
updated-dependencies:
- dependency-name: actions/download-artifact
  dependency-version: '6'
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>

Update dependabot.yml

Include "ci" prefix in dependabot commits

ci: Bump actions/upload-artifact from 4 to 5

Bumps [actions/upload-artifact](https://github.com/actions/upload-artifact) from 4 to 5.
- [Release notes](https://github.com/actions/upload-artifact/releases)
- [Commits](actions/upload-artifact@v4...v5)

---
updated-dependencies:
- dependency-name: actions/upload-artifact
  dependency-version: '5'
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>

ci: bump actions/checkout from 5 to 6

Bumps [actions/checkout](https://github.com/actions/checkout) from 5 to 6.
- [Release notes](https://github.com/actions/checkout/releases)
- [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md)
- [Commits](actions/checkout@v5...v6)

---
updated-dependencies:
- dependency-name: actions/checkout
  dependency-version: '6'
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>

Update code cov, include ruff (#91)

* Add issue test

* Bump version, fix broken badges in docs

* Update github action to use ruff, codecov token

* Add ruff dep

* Run ruff format, lint

fix(ci): bump checkout 5 to 6, skip codecov updates for bot

Bumps [actions/checkout](https://github.com/actions/checkout) from 5 to
6.
- [Release notes](https://github.com/actions/checkout/releases)
-
[Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md)
- [Commits](actions/checkout@v5...v6)

---
updated-dependencies:
- dependency-name: actions/checkout
  dependency-version: '6'
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>

fix(ci): skip codecov upload for dependabot PRs

ci: bump actions/download-artifact from 6 to 8

Bumps [actions/download-artifact](https://github.com/actions/download-artifact) from 6 to 8.
- [Release notes](https://github.com/actions/download-artifact/releases)
- [Commits](actions/download-artifact@v6...v8)

---
updated-dependencies:
- dependency-name: actions/download-artifact
  dependency-version: '8'
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>

ci: bump actions/upload-artifact from 5 to 7

Bumps [actions/upload-artifact](https://github.com/actions/upload-artifact) from 5 to 7.
- [Release notes](https://github.com/actions/upload-artifact/releases)
- [Commits](actions/upload-artifact@v5...v7)

---
updated-dependencies:
- dependency-name: actions/upload-artifact
  dependency-version: '7'
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>

ci: bump actions/download-artifact from 6 to 8

Bumps [actions/download-artifact](https://github.com/actions/download-artifact) from 6 to 8.
- [Release notes](https://github.com/actions/download-artifact/releases)
- [Commits](actions/download-artifact@v6...v8)

---
updated-dependencies:
- dependency-name: actions/download-artifact
  dependency-version: '8'
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
catanzaromj added a commit that referenced this pull request Mar 7, 2026
@dependabot @catanzaromj
ci: bump actions/download-artifact from 6 to 8 (#99)
3a9afe7 
* Add issue test

* Remove modificatin of ds

* Bump version, fix broken badges in docs

* Bump version, add tests

* Modify test for matching shape

* Remove old travis badge

* ci: Bump actions/download-artifact from 5 to 6

Bumps [actions/download-artifact](https://github.com/actions/download-artifact) from 5 to 6.
- [Release notes](https://github.com/actions/download-artifact/releases)
- [Commits](actions/download-artifact@v5...v6)

---
updated-dependencies:
- dependency-name: actions/download-artifact
  dependency-version: '6'
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>

Update dependabot.yml

Include "ci" prefix in dependabot commits

ci: Bump actions/upload-artifact from 4 to 5

Bumps [actions/upload-artifact](https://github.com/actions/upload-artifact) from 4 to 5.
- [Release notes](https://github.com/actions/upload-artifact/releases)
- [Commits](actions/upload-artifact@v4...v5)

---
updated-dependencies:
- dependency-name: actions/upload-artifact
  dependency-version: '5'
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>

ci: bump actions/checkout from 5 to 6

Bumps [actions/checkout](https://github.com/actions/checkout) from 5 to 6.
- [Release notes](https://github.com/actions/checkout/releases)
- [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md)
- [Commits](actions/checkout@v5...v6)

---
updated-dependencies:
- dependency-name: actions/checkout
  dependency-version: '6'
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>

Update code cov, include ruff (#91)

* Add issue test

* Bump version, fix broken badges in docs

* Update github action to use ruff, codecov token

* Add ruff dep

* Run ruff format, lint

fix(ci): bump checkout 5 to 6, skip codecov updates for bot

Bumps [actions/checkout](https://github.com/actions/checkout) from 5 to
6.
- [Release notes](https://github.com/actions/checkout/releases)
-
[Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md)
- [Commits](actions/checkout@v5...v6)

---
updated-dependencies:
- dependency-name: actions/checkout
  dependency-version: '6'
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>

fix(ci): skip codecov upload for dependabot PRs

ci: bump actions/download-artifact from 6 to 8

Bumps [actions/download-artifact](https://github.com/actions/download-artifact) from 6 to 8.
- [Release notes](https://github.com/actions/download-artifact/releases)
- [Commits](actions/download-artifact@v6...v8)

---
updated-dependencies:
- dependency-name: actions/download-artifact
  dependency-version: '8'
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>

ci: bump actions/upload-artifact from 5 to 7

Bumps [actions/upload-artifact](https://github.com/actions/upload-artifact) from 5 to 7.
- [Release notes](https://github.com/actions/upload-artifact/releases)
- [Commits](actions/upload-artifact@v5...v7)

---
updated-dependencies:
- dependency-name: actions/upload-artifact
  dependency-version: '7'
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>

ci: bump actions/download-artifact from 6 to 8

Bumps [actions/download-artifact](https://github.com/actions/download-artifact) from 6 to 8.
- [Release notes](https://github.com/actions/download-artifact/releases)
- [Commits](actions/download-artifact@v6...v8)

---
updated-dependencies:
- dependency-name: actions/download-artifact
  dependency-version: '8'
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>

* Bump to v7

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: Michael Catanzaro <catanzaromj@pm.me>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@catanzaromj