-
Notifications
You must be signed in to change notification settings - Fork 220
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
feat: libsql experiment #2178
feat: libsql experiment #2178
Conversation
markphelps
commented
Sep 30, 2023
•
edited
Loading
edited
- libsql support
- update to go 1.21
Codecov Report
@@ Coverage Diff @@
## main #2178 +/- ##
==========================================
+ Coverage 70.84% 70.92% +0.08%
==========================================
Files 77 77
Lines 7307 7334 +27
==========================================
+ Hits 5177 5202 +25
- Misses 1831 1833 +2
Partials 299 299
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
@@ -36,12 +36,39 @@ func TestOpen(t *testing.T) { | |||
{ | |||
name: "sqlite url", | |||
cfg: config.DatabaseConfig{ | |||
URL: "file:flipt.db", | |||
URL: "file:/flipt.db", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is this backward compatible? ie does it also work with file:flipt.db
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@markphelps Yes it seems to work with file:flipt.db
as well.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can you add a test just to be sure?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@markphelps Yup!
Uffizzi Ephemeral Environment
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks good to me. Only one question regarding the added Dockerfile.dev file.
@@ -0,0 +1,26 @@ | |||
FROM golang:1.21-alpine3.18 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this one still needed?
I can't remember how that conversation ended.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i think it still is for the dev docker-compose to not build the UI in the Go binary