FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

gh-108278: Deprecate passing the first param of sqlite3.Connection callback APIs by keyword by erlend-aasland · Pull Request #108632 · python/cpython · GitHub

  1. python
  2. cpython

Repository navigation

gh-108278: Deprecate passing the first param of sqlite3.Connection callback APIs by keyword - #108632

Merged
erlend-aasland merged 3 commits into
python:mainfrom
erlend-aasland:sqlite/cleanup-callback-apis
Aug 29, 2023
ConversationCommits (3)ChecksFiles changed
Merged

gh-108278: Deprecate passing the first param of sqlite3.Connection callback APIs by keyword#108632
erlend-aasland merged 3 commits into
python:mainfrom
erlend-aasland:sqlite/cleanup-callback-apis

Conversation

erlend-aasland commented Aug 29, 2023 •
edited by github-actions Bot
Loading

Copy link
Copy Markdown
Contributor

Deprecate passing the callback callable by keyword for the following
sqlite3.Connection APIs:

  • set_authorizer(authorizer_callback)
  • set_progress_handler(progress_handler, ...)
  • set_trace_callback(trace_callback)

The affected parameters will become positional-only in Python 3.15.

📚 Documentation preview 📚: https://cpython-previews--108632.org.readthedocs.build/

pythongh-108278: Deprecate passing the first param of sqlite3.Connect…
a207531 
…ion callback APIs by keyword

Deprecate passing the callback callable by keyword for the following
sqlite3.Connection APIs:

- set_authorizer(authorizer_callback)
- set_progress_handler(progress_handler, ...)
- set_trace_callback(trace_callback)

The affected parameters will become positional-only in Python 3.15.
bedevere-bot mentioned this pull request Aug 28, 2023
Closed

erlend-aasland commented Aug 29, 2023

Copy link
Copy Markdown
Contributor Author

Also fixup the indent of the versionchanged directives added in gh-108281

serhiy-storchaka approved these changes Aug 29, 2023
View reviewed changes

serhiy-storchaka left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

LGTM, but should not n in set_progress_handler() be positional-only too?

erlend-aasland commented Aug 29, 2023

Copy link
Copy Markdown
Contributor Author

LGTM, but should not n in set_progress_handler() be positional-only too?

I am not sure. I left a comment regarding this on the issue.

serhiy-storchaka commented Aug 29, 2023

Copy link
Copy Markdown
Member

I am not sure too.

erlend-aasland commented Aug 29, 2023

Copy link
Copy Markdown
Contributor Author

I am not sure too.

We can always deprecate passing n by keyword at a later time; either later in 3.13 development, or in a following release.

erlend-aasland commented Aug 29, 2023

Copy link
Copy Markdown
Contributor Author

I posted about this on Discourse, and got encouraging reactions from Brett Cannon and Berker Peksag. (Berker is also an sqlite3 code owner.) Together with Serhiy's approval, I intend to land this.

It is still very early in the 3.13 development phase, so we have plenty of time to adjust or even revert this feature.

Thanks for raising the discussions on the issue, and thanks for the review, Serhiy!

erlend-aasland merged commit 0b0c1d0 into python:main Aug 29, 2023
erlend-aasland deleted the sqlite/cleanup-callback-apis branch August 29, 2023 20:02

This comment was marked as off-topic.

Sign in to view

This comment was marked as off-topic.

Sign in to view
carljm added a commit to carljm/cpython that referenced this pull request Aug 30, 2023
Merge branch 'main' into icompexc
37377e6 
* main:
  pythongh-108520: Fix bad fork detection in nested multiprocessing use case (python#108568)
  pythongh-108590: Revert pythongh-108657 (commit 400a1ce) (python#108686)
  pythongh-108494: Argument Clinic: Document how to generate code that uses the limited C API (python#108584)
  Document Python build requirements (python#108646)
  pythongh-101100: Fix Sphinx warnings in the Logging Cookbook (python#108678)
  Fix typo in multiprocessing docs (python#108666)
  pythongh-108669: unittest: Fix documentation for TestResult.collectedDurations (python#108670)
  pythongh-108590: Fix sqlite3.iterdump for invalid Unicode in TEXT columns (python#108657)
  Revert "pythongh-103224: Use the realpath of the Python executable in `test_venv` (pythonGH-103243)" (pythonGH-108667)
  pythongh-106320: Remove private _Py_ForgetReference() (python#108664)
  Mention Ellipsis pickling in the docs (python#103660)
  Revert "Use non alternate name for Kyiv (pythonGH-108533)" (pythonGH-108649)
  pythongh-108278: Deprecate passing the first param of sqlite3.Connection callback APIs by keyword (python#108632)
  pythongh-108455: peg_generator: install two stubs packages before running mypy (python#108637)
  pythongh-107801: Improve the accuracy of io.IOBase.seek docs (python#108268)
gpshead mentioned this pull request Sep 30, 2024
Merged
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Show hidden characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

berkerpeksag Awaiting requested review from berkerpeksag
serhiy-storchaka Awaiting requested review from serhiy-storchaka

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

Back | FazBrowse Home | New Git URL