Skip to content

gh-108364: Simplify quoting values and identifiers in sqlite3's iterdump() #108472

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

Jump to bottom
Merged
merged 3 commits into from
Aug 25, 2023
Merged

gh-108364: Simplify quoting values and identifiers in sqlite3's iterdump() #108472

merged 3 commits into from
Aug 25, 2023

Conversation

felixxm
Copy link
Contributor

@felixxm felixxm commented Aug 25, 2023 •
edited by bedevere-bot
Loading

@felixxm felixxm mentioned this pull request Aug 25, 2023
Merged
felixxm
felixxm commented Aug 25, 2023
View reviewed changes
@erlend-aasland
Copy link
Contributor

This is a nice cleanup. I think we should add this as a separate PR, instead of as a part of #108340.

@erlend-aasland erlend-aasland changed the title Simplify quoting values/identifiers in sqlite3.dump._iter dump(). gh-108364: Simplify quoting values and identifiers in sqlite3's iterdump() Aug 25, 2023
@bedevere-bot bedevere-bot mentioned this pull request Aug 25, 2023
Closed
@felixxm felixxm force-pushed the sqlite-dump-readable branch from 333c0f5 to 2423154 Compare August 25, 2023 09:42
@felixxm
Copy link
Contributor Author

felixxm commented Aug 25, 2023

This is a nice cleanup. I think we should add this as a separate PR, instead of as a part of #108340.

Agreed. #108340 could use new hooks so I would merge this PR first.

@erlend-aasland
Copy link
Contributor

Agreed. #108340 could use new hooks so I would merge this PR first.

Sounds good to me.

@erlend-aasland
Copy link
Contributor

Seems main was recently broken by #107988; we'll have to wait for that issue to be resolved before landing.

@erlend-aasland erlend-aasland enabled auto-merge (squash) August 25, 2023 12:05
@erlend-aasland erlend-aasland merged commit 75903f2 into python:main Aug 25, 2023
@felixxm felixxm deleted the sqlite-dump-readable branch August 25, 2023 19:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@erlend-aasland erlend-aasland erlend-aasland approved these changes

@berkerpeksag berkerpeksag Awaiting requested review from berkerpeksag berkerpeksag is a code owner

Assignees
No one assigned
Labels
skip news
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@felixxm @erlend-aasland @bedevere-bot