Skip to content

Use String interpolation in SQLiteConnection data source name#431

Closed
kbeaugrand wants to merge 5 commits intomicrosoft:mainfrom
kbeaugrand:fix-sqlite-datasource-name
Closed

Use String interpolation in SQLiteConnection data source name#431
kbeaugrand wants to merge 5 commits intomicrosoft:mainfrom
kbeaugrand:fix-sqlite-datasource-name

Conversation

@kbeaugrand
Copy link
Copy Markdown
Contributor

@kbeaugrand kbeaugrand commented Apr 12, 2023

Motivation and Context

Fix #430

SQLite database is not created with the expected name.
Instead of the expected file name, we get a file name "{filename}" created in the current directory.
This issue occurs because the argumement is not passed correctly to the format function.

Description

The change only modify the SQLiteConnection instance by using the good string interpolation with the argument.

Contribution Checklist

@kbeaugrand kbeaugrand changed the title Fix #430 Use String interpolation in SQLiteConnection data source name Apr 12, 2023
@kbeaugrand kbeaugrand marked this pull request as ready for review April 12, 2023 21:00
adrianwyatt
adrianwyatt previously approved these changes Apr 13, 2023
@adrianwyatt adrianwyatt enabled auto-merge (squash) April 13, 2023 01:35
@adrianwyatt adrianwyatt added PR: ready to merge PR has been approved by all reviewers, and is ready to merge. PR: main labels Apr 13, 2023
@adrianwyatt adrianwyatt self-assigned this Apr 13, 2023
@awharrison-28
Copy link
Copy Markdown
Contributor

@adrianwyatt this issue will be resolved by #425

auto-merge was automatically disabled April 13, 2023 05:19

Head branch was pushed to by a user without write access

@awharrison-28 awharrison-28 added the duplicate This issue or pull request already exists label Apr 13, 2023
adrianwyatt
adrianwyatt previously approved these changes Apr 13, 2023
@adrianwyatt
Copy link
Copy Markdown
Contributor

Dupe with #425

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

duplicate This issue or pull request already exists PR: ready to merge PR has been approved by all reviewers, and is ready to merge.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Bad SQLite database name Assignees: kbeaugrand Labels:

3 participants