-
Notifications
You must be signed in to change notification settings - Fork 12.8k
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: Pylint errors on master #27710
fix: Pylint errors on master #27710
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #27710 +/- ##
==========================================
+ Coverage 69.77% 69.89% +0.11%
==========================================
Files 1911 1911
Lines 75056 75060 +4
Branches 8362 8362
==========================================
+ Hits 52374 52461 +87
+ Misses 20630 20547 -83
Partials 2052 2052
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
@michael-s-molina do we know what caused this to become a problem? |
@betodealmeida regarding your comment, the only explanation I can think of is that either for the PR which bumped
* Per #27574 we had previously identified issues were certain GitHub checks weren't marked as required. |
The workflow at .github/workflows/superset-python-misc.yml will only run if files in |
SUMMARY
This PR fixes the following Pylint errors on
master
:TESTING INSTRUCTIONS
CI should be sufficient.
ADDITIONAL INFORMATION