-
-
Notifications
You must be signed in to change notification settings - Fork 32.4k
gh-77617: Add interactive shell for sqlite3 #95026
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
Merged
Merged
Changes from 1 commit
Commits
Show all changes
37 commits
Select commit
Hold shift + click to select a range
def2e81
Add proof-of-concept REPL
1f01905
Add NEWS
erlend-aasland 86a4566
Remove redundant __name__ condition
erlend-aasland d04b6f3
Merge branch 'main' into sqlite-shell
erlend-aasland 896834c
Add -v argument for dumping SQLite version
erlend-aasland 4ba731a
Address review: close connection explicitly
erlend-aasland 814cced
Merge branch 'main' into sqlite-shell
erlend-aasland 8179a68
Add docs
erlend-aasland 91a77b7
Remove copyright, credits, and license cli commands
erlend-aasland 2f27a04
Document how to quit
erlend-aasland ae55e8e
Fix sphinx option refs by moving the footnote up to where it belongs,…
erlend-aasland 91e1ec9
Merge branch 'main' into sqlite-shell
erlend-aasland 024cd90
Address review: iterate over cursor
dee441a
Address review: repr iso. f-string
ec04ea3
Address review: pass connection iso. path
erlend-aasland 9b79230
Address review: use argparse version trick
erlend-aasland 74fb53d
Address review: use . prefix for commands, and sys.exit for quitting
erlend-aasland 1d62ccb
Address review: reduce indent level
erlend-aasland 340b896
Add filename and sql args a la sqlite3 cli
erlend-aasland d906361
Fix constructor
erlend-aasland 43c535c
Sync with main
erlend-aasland e728e17
Address Serhiy's second round of review
erlend-aasland e2e24b8
Remove useless function and lambda params
erlend-aasland 294acb9
Move CLI docs to Reference
erlend-aasland 3cfff7f
Improve NEWS and add What's New
erlend-aasland 82c753c
Partially address Kumar's review: use narrow exception
erlend-aasland 36454ab
Partially address more of Kumar's review: add guard
erlend-aasland 625e2e3
Use pattern matching
erlend-aasland f624873
Revert "Partially address more of Kumar's review: add guard"
erlend-aasland df75f5f
Print errors to stderr iso. stdout
erlend-aasland 44c2571
Non-zero exit for SQL errors passed from command line
erlend-aasland d640458
Add tests
erlend-aasland 0997004
Simplify interactive tests
erlend-aasland 7ba91e7
Use -Xutf8
erlend-aasland b74e010
Fix argument order
erlend-aasland 903b867
Address review: move cleanups up
erlend-aasland 796ba16
Last adjustment: inline runsql()
erlend-aasland File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Address review: use . prefix for commands, and sys.exit for quitting
- Loading branch information
commit 74fb53d30542fb32732bc64ef4fddae65966db5d
There are no files selected for viewing
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
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
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.