Skip to content

Conversation

@lucydodo
Copy link
Member

@lucydodo lucydodo commented Dec 9, 2023

Related: #3357

Since this PR changes include the external projects' library to our project, so I've added a license disclaimer.
I also left a PR for the same reason, because I think it's only fair that both owners review it before committing it right away.

@nalgeon You are the owner of the project we want to include, so I ping you together. :)

Thank you.

@lucydodo
Copy link
Member Author

lucydodo commented Dec 9, 2023

https://www.youtube.com/watch?v=FM5TSX9JRA8
I've also prepared a simple demostration video that loads and uses the newly included sqlean library.

GitHub can't upload video files larger than 10MB, so I inevitably upload to YouTube.

Copy link
Member

@justinclift justinclift left a comment

Choose a reason for hiding this comment

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

Looks good to me. 😄

@nalgeon
Copy link

nalgeon commented Dec 9, 2023

Looks great!

@justinclift justinclift merged commit 96686a7 into master Dec 9, 2023
@justinclift justinclift deleted the fix-3357 branch December 9, 2023 18:52
@justinclift
Copy link
Member

Cool. k, I've just hit the Merge button, so lets see if it all works properly... 😄

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants