fsprojects / FSharp.Data

F# Data: Library for Data Access
https://fsprojects.github.io/FSharp.Data
Other
806 stars 288 forks source link

Change the namespace of the public "InferenceMode" enum #1470

Closed mlaily closed 1 year ago

mlaily commented 1 year ago

This is a breaking change, but I feel like it's needed...

Using FSharp.Data.Runtime.StructuralInference was a mistake (sorry, my bad). It should not be necessary to reference that namespace just to initialize a type provider...

I moved the enum to the FSharp.Data namespace instead, the same as the providers so it's readily available:

I have no idea how much this new-ish enum is used in the wild, but since it's optional and a bit obscure, it might not impact too many users...

Since the next version will be the new major version 6.x and it's still currently in beta, I think this is a good opportunity to do this kind of thing now.