Skip to content

[WIP] More docs and tests - #11

Merged
espottesmith merged 6 commits into
mainfrom
docs_and_tests
Sep 25, 2025
Merged

espottesmith merged 6 commits into
mainfrom
docs_and_tests

Conversation

@espottesmith

Copy link
Copy Markdown
Member

Description

PR #6 added a significant body of tests but notably left transforms.jl untested. This PR will fill that gap and (hopefully) significantly improve test coverage.

Dependency and version changes

N/A

Change features

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Refactor (change, which could be breaking or non-breaking, that mainly involves reorganizing code)
  • This change requires a documentation update
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Other (please describe below)

Add tests

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas
  • I have added tests that prove my fix is effective or that my feature works
  • I have run tests locally, and all tests passed
  • My changes generate no (new) warnings

@codecov-commenter

codecov-commenter commented Sep 24, 2025 •

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 94.31818% with 5 lines in your changes missing coverage. Please review.
✅ Project coverage is 92.19%. Comparing base (500d75f) to head (aa9d04a).
⚠️ Report is 4 commits behind head on main.

Files with missing lines Patch % Lines
src/core/transform.jl 94.31% 5 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##             main      #11       +/-   ##
===========================================
+ Coverage   62.58%   92.19%   +29.61%     
===========================================
  Files           8        8               
  Lines        1983     1960       -23     
===========================================
+ Hits         1241     1807      +566     
+ Misses        742      153      -589     

☔ 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.

@espottesmith
espottesmith merged commit a963eda into main Sep 25, 2025
3 checks passed
@espottesmith
espottesmith deleted the docs_and_tests branch September 25, 2025 08:17
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.

2 participants