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
Make OPENLINEAGE_DISABLED case insensitive #1705
Make OPENLINEAGE_DISABLED case insensitive #1705
Conversation
In the python client and Airflow plugin, check OPENLINEAGE_DISABLED in a case insensitive way. Signed-off-by: Jed Cunningham <66968678+jedcunningham@users.noreply.github.com>
("true", True), | ||
("True", True), | ||
("TRUE", True), | ||
("false", False), | ||
("False", False), | ||
("FALSE", False), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If this is going to come from an environment variable, does that imply it's always a string? Or can these vars also come in as a True or False boolean?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry, meant to comment on that. Yes, getenv always returns a string.
Codecov Report
📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more @@ Coverage Diff @@
## main #1705 +/- ##
============================================
+ Coverage 83.34% 85.35% +2.00%
============================================
Files 89 69 -20
Lines 4006 3668 -338
Branches 29 0 -29
============================================
- Hits 3339 3131 -208
+ Misses 635 537 -98
+ Partials 32 0 -32 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
Great job! Congrats on your first merged pull request in OpenLineage! |
In the python client and Airflow plugin, check OPENLINEAGE_DISABLED in a case insensitive way. Signed-off-by: Jed Cunningham <66968678+jedcunningham@users.noreply.github.com>
Problem
It can be surprising that OPENLINEAGE_DISABLED is case sensitive.
Closes: #1704
Solution
In the python client and Airflow plugin, check OPENLINEAGE_DISABLED in a case insensitive way.
Checklist
SPDX-License-Identifier: Apache-2.0
Copyright 2018-2023 contributors to the OpenLineage project