Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add Scilpy version - part 2 #1106

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Conversation

elyz081
Copy link

@elyz081 elyz081 commented Dec 12, 2024

Fix issue #760

Part 2 of adding the Scilpy version to the scripts

@pep8speaks
Copy link

pep8speaks commented Dec 12, 2024

Hello @elyz081, Thank you for updating !

There are currently no PEP 8 issues detected in this Pull Request. Cheers! 🍻

Comment last updated at 2024-12-12 20:33:14 UTC

elyz0801 added 2 commits December 12, 2024 15:17
Copy link

codecov bot commented Dec 12, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 69.61%. Comparing base (e256482) to head (b2da519).

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1106      +/-   ##
==========================================
+ Coverage   69.46%   69.61%   +0.15%     
==========================================
  Files         447      447              
  Lines       24052    24077      +25     
  Branches     3291     3291              
==========================================
+ Hits        16708    16762      +54     
+ Misses       5946     5916      -30     
- Partials     1398     1399       +1     
Components Coverage Δ
Scripts 70.31% <100.00%> (+0.05%) ⬆️
Library 68.68% <100.00%> (+0.28%) ⬆️

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