Skip to content
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

Add lsqlite3 addon #113

Merged
merged 2 commits into from
Feb 28, 2025
Merged

Add lsqlite3 addon #113

merged 2 commits into from
Feb 28, 2025

Conversation

KaitlynEthylia
Copy link
Contributor

Definition files for LuaSQLite3

https://lua.sqlite.org

Copy link
Collaborator

@carsakiller carsakiller left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The submodule itself has not been committed. Please make sure you are using git submodule add to add the submodule and it is being committed, not just the changes to .gitmodules.

@carsakiller carsakiller added the New Addon Related to adding a new addon label Feb 4, 2024
@carsakiller
Copy link
Collaborator

The StyLua requirement has been removed. Please let me know if the definitions are still up-to-date and if you would like this merged.

@KaitlynEthylia
Copy link
Contributor Author

luasqlite doesn't appear to have been updated since I first made this, so yes it's still up to date, but as I'm not using the library any more, I have very little interest in maintaining the addon should the library update and change its API

Copy link
Collaborator

@carsakiller carsakiller left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Going to go ahead with merging. If in the future this does need updating, hopefully someone will be able to carry the torch 🙂

@carsakiller carsakiller merged commit b57b1e9 into LuaLS:main Feb 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
New Addon Related to adding a new addon
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants