Review version policy Newtonsoft.Json

[yvesgoeleven]

Connects to https://github.com/Particular/NServiceBus.Azure/issues/318

Current: Fixed Major Range

Used in

• NServiceBus.Azure.Transports.WindowsAzureStorageQueues: Range [9.0.0, 10.0.0)
• NServiceBus.Persistence.AzureStorage: Range [9.0.0, 10.0.0)
• NServiceBus.Persistence.ServiceFabric: Range [10.0.0, 11.0.0)

[yvesgoeleven]

Suggestion: Open Major Range [9.0.0, )

[boblangley]

Agreed, Open Major Range 👍

[Scooletz]

It might hit us back but 👍 is better than blocking people from bumping this up.

[danielmarbach]

In the fence on this one. Since sometimes we hook into the guts of the serializer this seems to be too dangerous since those interfaces can change in a major. I'd rather deal with minor version bumps on all our azure offerings. So 👎

[yvesgoeleven]

@danielmarbach where do we do this?

[yvesgoeleven]

@danielmarbach how about a closed range [9.0.0.0, 11.0.0.0) that we extend after testing any new major coming from newtonsoft?

[yvesgoeleven]

Or as @SimonCropp suggested, replace it with https://msdn.microsoft.com/en-us/library/system.runtime.serialization.json.datacontractjsonserializer.aspx

[danielmarbach]

Or ILMerge repack newtonsoft json

On Thu, Jun 8, 2017 at 2:33 PM Yves Goeleven notifications@github.com wrote:

Or as @SimonCropp https://github.com/simoncropp suggested, replace it with https://msdn.microsoft.com/en-us/library/system.runtime.serialization.json.datacontractjsonserializer.aspx

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/Particular/NServiceBus.Azure/issues/300#issuecomment-307089385, or mute the thread https://github.com/notifications/unsubscribe-auth/AAKosjCTNYhFU2EeF6-ViQSCZUXmrcUEks5sB-oZgaJpZM4NpTLO .

[yvesgoeleven]

Will be promoted to platdev

[Scooletz]

Could you link the platdev issue @yvesgoeleven ?