Title: Remove git.util.NullHandler by EliahKagan · Pull Request #1807 · gitpython-developers/GitPython · GitHub
Open Graph Title: Remove git.util.NullHandler by EliahKagan · Pull Request #1807 · gitpython-developers/GitPython
X Title: Remove git.util.NullHandler by EliahKagan · Pull Request #1807 · gitpython-developers/GitPython
Description: This is not related to the issue of when a noop handler should actually be used, so it is independent of #1806. The NullHandler class in git.util was added when merging #300, to allow a noop handler to be used on Python 2.6, since the standard library logging.NullHandler class was added in Python 2.7. When introduced in d1a9a23, the git.util.NullHandler class was also patched into the logging module, but that has no longer been done since 2fced2e (#979), nor does GitPython make other use of it. git/util.py defines __all__, which does not list NullHandler. As NullHandler is not otherwise documented to be public, it is a bug for any code outside GitPython to use it, and removing it is not a breaking change. So there should be no need for a deprecation period or to wait until the next major version of GitPython. For some reason I had not, at first, thought to check if it was in __all__, and assumed it was. The first commit here reflects that mistake, documenting it as deprecated and why the standard library NullHandler is preferable, and having it issue DeprecationWarning when constructed. Although I wouldn't have written that if I'd been paying proper attention, I decided to keep it, since the documented difference between it and logging.NullHandler may be useful to see in the history. However, I'd be happy to rebase to remove that commit and update the other commit's message accordingly, if desired.
Open Graph Description: This is not related to the issue of when a noop handler should actually be used, so it is independent of #1806. The NullHandler class in git.util was added when merging #300, to allow a noop handle...
X Description: This is not related to the issue of when a noop handler should actually be used, so it is independent of #1806. The NullHandler class in git.util was added when merging #300, to allow a noop handle...
Opengraph URL: https://github.com/gitpython-developers/GitPython/pull/1807
X: @github
Domain: github.com
| route-pattern | /:user_id/:repository/pull/:id/checks(.:format) |
| route-controller | pull_requests |
| route-action | checks |
| fetch-nonce | v2:7dc83901-91a6-5a72-b814-89f8bf46189d |
| current-catalog-service-hash | 87dc3bc62d9b466312751bfd5f889726f4f1337bdff4e8be7da7c93d6c00a25a |
| request-id | D1D2:283BF5:D99003:125031C:69687F88 |
| html-safe-nonce | 3dad502c17c79a43ce51f1cb1f77524b62f401addc16169e1caadf1f392a879b |
| visitor-payload | eyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiJEMUQyOjI4M0JGNTpEOTkwMDM6MTI1MDMxQzo2OTY4N0Y4OCIsInZpc2l0b3JfaWQiOiIzMDkxNTExOTU5NTc2MTQ5ODk2IiwicmVnaW9uX2VkZ2UiOiJpYWQiLCJyZWdpb25fcmVuZGVyIjoiaWFkIn0= |
| visitor-hmac | f2afbc63cccb3aec9a6e0b582bd160c15631271578c330fbf7ba59e1b609beb9 |
| hovercard-subject-tag | pull_request:1692734759 |
| github-keyboard-shortcuts | repository,pull-request-list,pull-request-conversation,pull-request-files-changed,checks,copilot |
| google-site-verification | Apib7-x98H0j5cPqHWwSMm6dNU4GmODRoqxLiDzdx9I |
| octolytics-url | https://collector.github.com/github/collect |
| analytics-location | / |
| fb:app_id | 1401488693436528 |
| apple-itunes-app | app-id=1477376905, app-argument=https://github.com/gitpython-developers/GitPython/pull/1807/checks |
| twitter:image | https://avatars.githubusercontent.com/u/1771172?s=400&v=4 |
| twitter:card | summary_large_image |
| og:image | https://avatars.githubusercontent.com/u/1771172?s=400&v=4 |
| og:image:alt | This is not related to the issue of when a noop handler should actually be used, so it is independent of #1806. The NullHandler class in git.util was added when merging #300, to allow a noop handle... |
| og:site_name | GitHub |
| og:type | object |
| hostname | github.com |
| expected-hostname | github.com |
| None | 50f46dc2d6192249fd8ebf20e76c800f4f2596d4a5f3ab63dd63a754df154f54 |
| turbo-cache-control | no-preview |
| go-import | github.com/gitpython-developers/GitPython git https://github.com/gitpython-developers/GitPython.git |
| octolytics-dimension-user_id | 503709 |
| octolytics-dimension-user_login | gitpython-developers |
| octolytics-dimension-repository_id | 1126087 |
| octolytics-dimension-repository_nwo | gitpython-developers/GitPython |
| octolytics-dimension-repository_public | true |
| octolytics-dimension-repository_is_fork | false |
| octolytics-dimension-repository_network_root_id | 1126087 |
| octolytics-dimension-repository_network_root_nwo | gitpython-developers/GitPython |
| turbo-body-classes | logged-out env-production page-responsive full-width full-width-p-0 |
| disable-turbo | false |
| browser-stats-url | https://api.github.com/_private/browser/stats |
| browser-errors-url | https://api.github.com/_private/browser/errors |
| release | fef287f17234b4529a4b112a3d47fe8551e32ddd |
| ui-target | full |
| theme-color | #1e2327 |
| color-scheme | light dark |
Links:
Viewport: width=device-width