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

refactor: remove assert statement from non-test files #32

Merged
merged 2 commits into from
Mar 29, 2024

Conversation

deepsource-autofix[bot]
Copy link
Contributor

Usage of assert statement in application logic is discouraged. assert is removed with compiling to optimized byte code. Consider raising an exception instead. Ideally, assert statement should be used only in tests.

Usage of `assert` statement in application logic is discouraged. `assert` is removed with compiling to optimized byte code. Consider raising an exception instead. Ideally, `assert` statement should be used only in tests.
This commit fixes the style issues introduced in 540b33f according to the output
from isort and Yapf.

Details: #32
@clementpoiret clementpoiret merged commit 5fb1d84 into master Mar 29, 2024
6 of 8 checks passed
Copy link
Contributor

deepsource-io bot commented Mar 29, 2024

Here's the code health analysis summary for commits 3dcd9b0..5ac4253. View details on DeepSource ↗.

Analysis Summary

AnalyzerStatusSummaryLink
DeepSource Python LogoPython✅ Success
🎯 3 occurences resolved
View Check ↗

💡 If you’re a repository administrator, you can configure the quality gates from the settings.

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.

1 participant