search

NEventStore / NEventStore.Persistence.RavenDB

RavenDB Persistence Engine for NEventStore
MIT License
2 stars 10 forks source link

Upgrade to Raven 3.0 #10

Closed kblooie closed 9 years ago

Iridio commented 9 years ago

I have already done the migration, but it's necessary the framework 4.5. So before doing a PR I was waiting to know when NES will move to 4.5

kblooie commented 9 years ago

Ha, I did as well.

All of NES doesn't have to move to .NET 4.5, we just have to ensure that it has a 4.5 target along with 4.0 in the nuget package. I thought I added that a while back. If not, I'll add it today.

More importantly, this will be a breaking change for people using .NET 4.0 or running on Raven 2.5. The checkpoint api change you contributed (thanks a bunch by the way) could be important to people who are on 4.0 or raven 2.5. Early on, we considered releasing a v5 and v6 version of the raven package at the same time to tackle this. Which is why all the persistence engines got split out into their own repos.

Who thinks this is needed?

kblooie commented 9 years ago

Brain fart. I don't think we NEED to support .NET 4.5 in the core NEventStore package to be able to use it in the raven implementation. With multi-targeting support, it still builds.

kblooie commented 9 years ago

I'll leave this open until we decide if we should release an interim 5.5 with ravendb 2.5 and v4