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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix: Prevent warnings/errors when running meltano config <plugin> test
#6676
fix: Prevent warnings/errors when running meltano config <plugin> test
#6676
Conversation
馃懛 Deploy request for meltano pending review.Visit the deploys page to approve it
|
Codecov Report
@@ Coverage Diff @@
## main #6676 +/- ##
=======================================
Coverage 82.91% 82.91%
=======================================
Files 178 178
Lines 11360 11360
Branches 1553 1553
=======================================
Hits 9419 9419
Misses 1675 1675
Partials 266 266
馃摚 We鈥檙e building smart automated test selection to slash your CI/CD build times. Learn more |
Linter was complaining about docstrings in |
@ReubenFrankel lgtm! thanks for taking the time to add docstrings and fix the type annotations. I'll request review from other folks in the team in case I missed anything. |
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.
Looks great! Thanks @ReubenFrankel!
Resolves #6632 by waiting for subprocess to exit after terminating given a
RECORD
message received, instead of immediately returning from the method. Includes some other minor related fixes, since I was in the area. 馃槃Before fix
Python 3.7
Python 3.9
After fix
Python 3.7
Python 3.9
Checklist