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

add ; to CREATE TABLE statement for Copy Create statement by yfdyh000 · Pull Request #3877 · sqlitebrowser/sqlitebrowser · GitHub

add ; to CREATE TABLE statement for Copy Create statement - #3877

Open
yfdyh000 wants to merge 1 commit into
sqlitebrowser:masterfrom
yfdyh000:2025-02-25-pr1
Open

add ; to CREATE TABLE statement for Copy Create statement#3877
yfdyh000 wants to merge 1 commit into
sqlitebrowser:masterfrom
yfdyh000:2025-02-25-pr1

Conversation

Copy link
Copy Markdown
Contributor

Before this, if you copied and pasted the Database Structure - Table items multiple times and tried to execute SQL statements, you would get syntax errors due to missing ;.

Copy link
Copy Markdown
Member

Hi, it looks like you're generating quite a few PRs in a short space of time.

Thanks for your enthusiasm. 😄

Just to check something, are you using AI of some variety to help with generating these PRs?

If so, please don't use AI generated code in the PRs. It's probably fine to use AI to help locate potential bugs then investigate yourself, but AI generated code is not well understood from a licensing perspective.

Different countries around the world have different ideas and it'll probably be a few more years until a clearer understanding of legalities emerges. 😄

Copy link
Copy Markdown
Contributor Author

Hello, thank you for your comment. I manually handle the tracking, committing and testing of these PRs. AI Tools like GitHub Copilot are used occasionally for trivial queries and guidance, as well as for auto-completion within VS. If you think anything might be copyrighted, please let me know. Otherwise, I think they are just simple logical snippets and not a cause for concern. 😊

By the way, I still keep track of over 50 issues for this project (accumulated over time and out of passion), but organizing, fixing, proposing or discussing them remains a challenging task. I'm not sure of the best practices, as the project is not very active.

Copy link
Copy Markdown
Member

I think they are just simple logical snippets and not a cause for concern.

No worries, was just checking to be careful. 😄

mgrojo commented May 24, 2025

Copy link
Copy Markdown
Member

I'm not sure about this simple change, because we might break some table creation statement, while the ';' is easy to add by the user. Can we be sure it is always safe to add the semicolon?

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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants


Back | FazBrowse Home | New Git URL