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

Fix dry-run with filter #65

Merged
merged 1 commit into from
May 30, 2023

Conversation

chris-schra
Copy link
Contributor

see #64

Disclaimer: I have NO IDEA of Python and really just changed that LoC - but it seems to work

@chris-schra chris-schra changed the title fix dry-run with filter Fix dry-run with filter May 30, 2023
@codecov
Copy link

codecov bot commented May 30, 2023

Codecov Report

Merging #65 (a45cd46) into main (7c3662e) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main      #65   +/-   ##
=======================================
  Coverage   93.27%   93.27%           
=======================================
  Files           1        1           
  Lines         223      223           
  Branches       54       54           
=======================================
  Hits          208      208           
  Misses          7        7           
  Partials        8        8           
Impacted Files Coverage Δ
main.py 93.27% <100.00%> (ø)

@sondrelg
Copy link
Member

Yeah this seems like a pretty safe one, so let's go ahead with it 🤞 Thanks @chris-schra!

@sondrelg sondrelg merged commit 2ebfab7 into snok:main May 30, 2023
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