Skip to content

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

Open
deepsource-autofix[bot] wants to merge 1 commit into
mainfrom
deepsource-autofix-908795c2
Open

refactor: remove assert statement from non-test files#4
deepsource-autofix[bot] wants to merge 1 commit into
mainfrom
deepsource-autofix-908795c2

Conversation

@deepsource-autofix

Copy link
Copy Markdown

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.
@deepsource-io

deepsource-io Bot commented Feb 24, 2025

Copy link
Copy Markdown

Here's the code health analysis summary for commits 1fac5ac..3873781. View details on DeepSource ↗.

Analysis Summary

AnalyzerStatusSummaryLink
DeepSource Python LogoPython❌ Failure
❗ 3 occurences introduced
🎯 7 occurences resolved
View Check ↗

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

@deepsource-development

Copy link
Copy Markdown

Here's the code health analysis summary for commits 1fac5ac..3873781. View details on DeepSource ↗.

Analysis Summary

AnalyzerStatusSummaryLink
DeepSource Python LogoPython❌ Failure
❗ 3 occurences introduced
🎯 7 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.

0 participants