Skip to content

Replace assert by if...raise in www package - #34249

Merged
hussein-awala merged 1 commit into
apache:mainfrom
hussein-awala:webserver_assertion
Sep 9, 2023
Merged

Replace assert by if...raise in www package#34249
hussein-awala merged 1 commit into
apache:mainfrom
hussein-awala:webserver_assertion

Conversation

@hussein-awala

Copy link
Copy Markdown
Member

assert statement is removed when the compilation is optimized, from the doc:

Assert statements are a convenient way to insert debugging assertions into a program:
assert_stmt ::= "assert"

The simple form, assert expression, is equivalent to
if debug:
if not expression: raise AssertionError

@hussein-awala
hussein-awala merged commit 7b4fc39 into apache:mainSep 9, 2023
@ephraimbuddyephraimbuddy added this to the Airflow 2.7.2 milestone Oct 3, 2023
@ephraimbuddyephraimbuddy added the type:misc/internal Changelog: Misc changes that should appear in change log label Oct 3, 2023
ephraimbuddy pushed a commit that referenced this pull request Oct 5, 2023
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:webserverWebserver related Issuestype:misc/internalChangelog: Misc changes that should appear in change log

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@hussein-awala@potiuk@ephraimbuddy