I'll start a new series to record my Xvc development progress. I think it's nice to have dialog between 🐢 and 🐇 to explain the decisions and progress. These will be published in my personal blog for now. In the future, we can have a dedicated blog for this.

🐇 LGTM. I think you can also link to commits here when they occur.

🐢 I think that's a bit too much work but PR's may be linked.

🐇 Ok. Let's start with the PRs. What do you work on these days?

🐢 I started fixing warnings in https://github.com/iesahin/xvc/issues/3 and now fixing missing documentation warnings.

🐇 Do you really think that we'll have a use for [RMNStore] in the future?

🐢 I'm not sure. Possibly any M-N relationship can be tracked by two 1-N relationships. Under the hood this is what RMNStore does actually.

🐇 Maybe we should just remove it.

🐢 I want to save it for the time being. If [xvc-ecs] will be used in other places, it may be needed.

🐇 Ok. Probably YAGNI, but let's keep it for the time being.

🐢 Writing documentation may be a debugging method too. I think I've found the reason for a bug here

🐇 Umm, how could it lead to a bug?

🐢 Insert mean new event happened. But when converting a VStore to an XvcStore, new events are not happening. We are simply converting from one store to another.

🐇 That means, they are added to XvcStore::current events, not XvcStore::previous , and when save_dir is called, they are saved.

🐢 Yeah, that was the bug. Time to time I was seeing duplicate JSON files.

🐇 Then maybe it's better to limit all insert usage.

🐢 Yeah, I should review that too.

🐇 Added a bug report .

🐢 I'm installing cargo-geiger to show that there is no unsafe code.

🐇 Do you think you'll maintain Xvc that way? It won't require any unsafe code?

🐢 Maintain the forbidden(unsafe) status? I think so. Xvc doesn't seem to need any unsafe blocks by itself. Most of the dependencies may need though.

🐇 Working on it. and will be merged today.

🐢 I worked about that s3cmd bug yesterday and it turned out that my modification on the configuration was the cause.

🐇 You were thinking that it's about R2 all along?

🐢 I should suspect with my custom config more.

🐇 Anyway, congratulations 🌠 that Xvc now has Cloudflare R2 support.