Agent removed. Deps on JToken removed.

Dzoukr / CosmoStore

F# Event store for Azure Cosmos DB, Table Storage, Postgres, LiteDB & ServiceStack

MIT License

168 stars 21 forks source link

Agent removed. Deps on JToken removed. #33

Closed kunjee17 closed 4 years ago

kunjee17 commented 4 years ago

@Dzoukr hey can you please check if Newtonsoft.Json can be updated to latest. Currently Tests are locked on version 11, I guess because of CosmoStore.

Not that is conflicting with Marten as it has pushed the deps.

Dzoukr commented 4 years ago

I think you can update deps for Test project with no problem (should not break anything) so keep free to update as needed.

Dzoukr commented 4 years ago

Looking at diff - are there really changes in CosmoStore validation or is it just formatting?

kunjee17 commented 4 years ago

@Dzoukr Just formatting. I was trying to generalize it. But then didn't did it.

Dzoukr commented 4 years ago

So this one can be closed since #14 is merged, right?

kunjee17 commented 4 years ago

Yes. It should be. But it is still showing changes. I'll have a look at it.

kunjee17 commented 4 years ago

There is only one minor change. Removed unused open. You can merge it or close it whatever you like.

@Dzoukr