search

microsoft / dbt-fabric

MIT License
79 stars 28 forks source link

support driver's ActiveDirectoryServicePrincipal option #75

Closed dataders closed 1 year ago

dataders commented 1 year ago

resolves: #76

I said this in #74, but this implementation leaves a bad taste in my mouth. ITT we shouldn't have to add yet another conditional statement. either we exhaustively model all possible configurations, or make the logic simpler.

prdpsvs commented 1 year ago

@dataders , i merged with 1.6.1 release and I am closing this PR without merging.