feat: enhance CI/CD, increase coverage, and fix extract_dataframe bugs #55
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Introduces a 'changes' job using dorny/paths-filter to conditionally trigger quality/test and docs jobs. This optimization prevents unnecessary CI runs when changes are unrelated (e.g., Python tests won't run for web-only changes, and vice-versa).
Adds pytest-cov to dev dependencies and configures pytest to report coverage for the src/loclean directory.
Removes uv.lock from .gitignore to ensure reproducible builds in CI/CD and across development environments.
Adds 'actions/checkout' to the 'changes' job to ensure 'dorny/paths-filter' has access to the git repository context.
Implements comprehensive tests for extract_dataframe, fixes Polars list type handling, resolves SQLite resource warnings, raises coverage threshold to 50%, and optimizes CI paths-filter.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR increases test coverage to >60%, fixes several bugs in dataframe extraction (specifically with Polars list types), and resolves SQLite resource warnings in the cache.