-
Notifications
You must be signed in to change notification settings - Fork 73
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
feat: Give users the option to log Python warnings #1294
feat: Give users the option to log Python warnings #1294
Conversation
Codecov Report
@@ Coverage Diff @@
## main #1294 +/- ##
==========================================
+ Coverage 85.19% 85.25% +0.05%
==========================================
Files 54 57 +3
Lines 4722 4740 +18
Branches 803 803
==========================================
+ Hits 4023 4041 +18
Misses 507 507
Partials 192 192
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
1df7d48
to
608c0cc
Compare
…tch-python-warning-for-deprecation-of-get_next_page_token
…tch-python-warning-for-deprecation-of-get_next_page_token
…tch-python-warning-for-deprecation-of-get_next_page_token
…tch-python-warning-for-deprecation-of-get_next_page_token
…tch-python-warning-for-deprecation-of-get_next_page_token
…tch-python-warning-for-deprecation-of-get_next_page_token
1405408
to
dcacc35
Compare
…tch-python-warning-for-deprecation-of-get_next_page_token
dcacc35
to
babdc83
Compare
This has been marked as stale because it is unassigned, and has not had recent activity. It will be closed after 21 days if no further activity occurs. If this should never go stale, please add the |
superseded by #768 |
This came up during Office Hours on 2022-12-21. Tap developers are only seeing the
get_next_page_token
deprecation warning in CI, which is not terrible, but it'd be nice if a warning was also logged during the tap execution.📚 Documentation preview 📚: https://meltano-sdk--1294.org.readthedocs.build/en/1294/