Skip to content

Minor style cleanup#133

Merged
brews merged 2 commits intomainfrom
style_cleanup
Aug 15, 2023
Merged

Minor style cleanup#133
brews merged 2 commits intomainfrom
style_cleanup

Conversation

@brews
Copy link
Copy Markdown
Member

@brews brews commented Aug 15, 2023

Updated ruff (as of #131) is failing a line of code for style in CI. This fixes the line to keep ruff happy.

Updated ruff is failing this line of code for style in CI. This fixes the line to keep ruff happy.
@brews brews self-assigned this Aug 15, 2023
@brews brews changed the title Clean up str type check for ruff check Minor style cleanup Aug 15, 2023
@codecov
Copy link
Copy Markdown

codecov Bot commented Aug 15, 2023

Codecov Report

Merging #133 (95ec0de) into main (199d60a) will not change coverage.
Report is 1 commits behind head on main.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main     #133   +/-   ##
=======================================
  Coverage   65.30%   65.30%           
=======================================
  Files          17       17           
  Lines        1842     1842           
=======================================
  Hits         1203     1203           
  Misses        639      639           
Files Changed Coverage Δ
src/dscim/preprocessing/input_damages.py 88.68% <100.00%> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@brews brews marked this pull request as ready for review August 15, 2023 18:13
@brews brews requested a review from kemccusker August 15, 2023 18:14
@brews brews merged commit 0f5f4c6 into main Aug 15, 2023
@brews brews deleted the style_cleanup branch August 16, 2023 19:46
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