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 output #56

Merged
merged 3 commits into from
Feb 14, 2023
Merged

Fix output #56

merged 3 commits into from
Feb 14, 2023

Conversation

lukasz-mitka
Copy link
Contributor

No description provided.

@sondrelg
Copy link
Member

Thanks @lukasz-mitka. Would you mind fixing the test as well?

@lukasz-mitka
Copy link
Contributor Author

lukasz-mitka commented Feb 14, 2023

Done. Didn't test it though. Out of time.

@codecov
Copy link

codecov bot commented Feb 14, 2023

Codecov Report

Merging #56 (9a06221) into main (2d1b1bd) will decrease coverage by 1.32%.
The diff coverage is 60.00%.

@@            Coverage Diff             @@
##             main      #56      +/-   ##
==========================================
- Coverage   94.47%   93.15%   -1.32%     
==========================================
  Files           1        1              
  Lines         217      219       +2     
  Branches       52       53       +1     
==========================================
- Hits          205      204       -1     
- Misses          6        7       +1     
- Partials        6        8       +2     
Impacted Files Coverage Δ
main.py 93.15% <60.00%> (-1.32%) ⬇️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@den-is
Copy link
Contributor

den-is commented Feb 14, 2023

You guys rock!!!

@sondrelg sondrelg merged commit 482ce28 into snok:main Feb 14, 2023
@lukasz-mitka lukasz-mitka deleted the patch-1 branch February 15, 2023 09:19
@lukasz-mitka
Copy link
Contributor Author

@sondrelg waiting for a release then!

@sondrelg
Copy link
Member

v2 released, finally. Thanks for all the great help @lukasz-mitka 👏

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