Skip to content

charge_array_fix#47

Merged
fletchapin merged 6 commits intomainfrom
bug_fix/charge_array
Dec 9, 2025
Merged

charge_array_fix#47
fletchapin merged 6 commits intomainfrom
bug_fix/charge_array

Conversation

@ThyanRevolter
Copy link
Contributor

@ThyanRevolter ThyanRevolter commented Dec 2, 2025

Pull request recommendations:

  • Name your pull request your-development-type/short-description. Ex: feature/gui
  • Link to any relevant issue in the PR description. Ex: Resolves [BUG:calculate_energy_cost has indexing bug #48]
  • Provide context of changes.
  • Provide relevant tests for your feature or bug fix.
  • Provide or update documentation for any feature added by your pull request.

Thanks for contributing!

@codecov-commenter
Copy link

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 91.00%. Comparing base (28ddcc0) to head (1ffd398).

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #47      +/-   ##
==========================================
- Coverage   91.00%   91.00%   -0.01%     
==========================================
  Files          10       10              
  Lines        1357     1356       -1     
==========================================
- Hits         1235     1234       -1     
  Misses        122      122              

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

@fletchapin fletchapin merged commit d86ca9c into main Dec 9, 2025
2 checks passed
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.

3 participants

Comments