Skip to content

Conversation

@Queyrouzec
Copy link
Contributor

Now works on the master branch

Updated IO in build file is now centralized to one variable so that it can be passed as a package argument later.

General writer and reader updates

Removed writer and reader from blob

General std updates

Updated Branch limit due to compile errors in personal use

@Queyrouzec
Copy link
Contributor Author

The error is a change in std.time. I'll check it and fix it later tonight or tomorrow.

@jozip jozip mentioned this pull request Jan 3, 2026
@vrischmann
Copy link
Owner

Hi,

sorry for not responding earlier.

Thank you for working on this. I'll try to do a first round of review at some point this week.

@jozip
Copy link

jozip commented Jan 7, 2026

I got a PR up on the fork with a fix for the time stuff: https://github.com/Queyrouzec/zig-sqlite/pull/1.diff

Use Io.Clock instead of time.Instant in tests
@Queyrouzec
Copy link
Contributor Author

Thanks for the help, and sorry for the delayed response. The std is getting a lot easier to understand, but I have no clue where anything is sometimes, and I'm not the best systems programmer. Looking through some of the std implementations, I think that there may also need to be a WASM test runner. I'm going to be messing with WASM some time this year, and I'll add it in then.

@jozip
Copy link

jozip commented Jan 9, 2026

Thanks for the help, and sorry for the delayed response. The std is getting a lot easier to understand, but I have no clue where anything is sometimes, and I'm not the best systems programmer. Looking through some of the std implementations, I think that there may also need to be a WASM test runner. I'm going to be messing with WASM some time this year, and I'll add it in then.

Everything is moving (and breaking) very fast on master right now and it's a challenge to keep up. I just think it's cool that more people get their hands dirty. :)

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