Skip to content

Dependencies - Remove unused, upgrade others#167

Merged
AbdealiLoKo merged 3 commits into
mainfrom
ashwin/deps
Apr 28, 2026
Merged

Dependencies - Remove unused, upgrade others#167
AbdealiLoKo merged 3 commits into
mainfrom
ashwin/deps

Conversation

@winwinashwin

Copy link
Copy Markdown
Contributor

No description provided.

@ashish16052

Copy link
Copy Markdown
Collaborator

did uv lock -u upgrade pytest and filelock? as per dependabot

@winwinashwin

Copy link
Copy Markdown
Contributor Author

filelock -> This dependency is no longer present, was coming from tox and pre-commit which I've handled case by case
pytest -> It's transitive coming from pytest-cov. Because it is a test dependency and we don't include it in package metadata I'm thinking we'll just dismiss it for now

@coveralls

coveralls commented Apr 28, 2026

Copy link
Copy Markdown

Coverage Report for CI Build 25040579091

Warning

No base build found for commit 485d5e0 on main.
Coverage changes can't be calculated without a base build.
If a base build is processing, this comment will update automatically when it completes.

Coverage: 95.11%

Details

  • Patch coverage: No coverable lines changed in this PR.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

Requires a base build to compare against. How to fix this →


Coverage Stats

Coverage Status
Relevant Lines: 5378
Covered Lines: 5115
Line Coverage: 95.11%
Coverage Strength: 9.44 hits per line

💛 - Coveralls

Comment thread pyproject.toml
prek is rust based and leaner without extra dependencies
We don't use tox anymore
@AbdealiLoKo AbdealiLoKo merged commit 1956003 into main Apr 28, 2026
14 checks passed
@AbdealiLoKo AbdealiLoKo deleted the ashwin/deps branch April 28, 2026 08:09
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.

4 participants