Skip to content

gh-61162: Clarify sqlite3 connection context manager docs #93890

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.

Sign up for GitHub

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

Merged
miss-islington merged 16 commits into from
Jun 19, 2022
+19 −4
Merged

gh-61162: Clarify sqlite3 connection context manager docs #93890

Changes from 1 commit
Commits
Show all changes
16 commits
Select commit Hold shift + click to select a range
672ce97
gh-61162: Clarify sqlite3 connection context manager docs
erlend-aasland Jun 16, 2022
61b31f9
Update Doc/library/sqlite3.rst
Jun 16, 2022
2434ba2
Address Adam's review: focus on the happy path
erlend-aasland Jun 16, 2022
71f2f45
Remove the now obsolete note
erlend-aasland Jun 16, 2022
76319b7
Clarify?
erlend-aasland Jun 16, 2022
20a100a
Update sqlite3.rst
Jun 16, 2022
28c435f
Add context manager anchor
Jun 16, 2022
6baac81
Use anchor in NEWS entry
Jun 16, 2022
8808fb9
Update Doc/library/sqlite3.rst
Jun 16, 2022
7237399
Add connection class link
Jun 16, 2022
6218dba
Make it more concise
Jun 16, 2022
32beb9f
Adjust wording and reflow for 79 chars
erlend-aasland Jun 17, 2022
a64fad4
Update Doc/library/sqlite3.rst
Jun 17, 2022
420c8ce
Object is singular
Jun 17, 2022
9fc3801
Reflow to 79 char limit
erlend-aasland Jun 17, 2022
b2ae22c
Finishing touch: Use neither...nor instead of does not...or
erlend-aasland Jun 19, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Address Adam's review: focus on the happy path
  • Loading branch information
@erlend-aasland
erlend-aasland committed Jun 16, 2022
commit 2434ba2be26b7838779e59ca8c24b5947dfd8624
11 changes: 6 additions & 5 deletions Doc/library/sqlite3.rst
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1434,12 +1434,13 @@ case-insensitively by name:
Using the connection as a context manager
Comment thread
erlend-aasland marked this conversation as resolved.
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

Connection objects can be used as context managers
that automatically commit or rollback transactions,
*if* there is an open transaction.
With an open transaction, connection objects can be used as context managers
that automatically commit or rollback transactions.
If the body of the ``with`` statement finishes without exceptions,
Comment thread
erlend-aasland marked this conversation as resolved.
Outdated
the transaction is committed.
Comment thread
erlend-aasland marked this conversation as resolved.
If an exception is raised and not caught, the transaction is rolled back.

If there is no open transaction, the context manager is a no-op.
Comment thread
erlend-aasland marked this conversation as resolved.
Outdated
In the event of an exception, the transaction is rolled back;
otherwise, the transaction is committed:

.. note::

Expand Down