Skip to content

Comments

Use package reference instead of project reference#2

Closed
virzak wants to merge 1 commit into0xced:mainfrom
virzak:package
Closed

Use package reference instead of project reference#2
virzak wants to merge 1 commit into0xced:mainfrom
virzak:package

Conversation

@virzak
Copy link

@virzak virzak commented Mar 1, 2024

This requires less effort for the user since they can simply reference a nuget package.

If you like the approach we can remove the nuget.config file and use nuget instead of myget.

@0xced
Copy link
Owner

0xced commented Jun 18, 2025

Hey Viktor, sorry I completely forgot about this pull request!

Note that I have now released Chisel. Getting rid of the Azure authentication libraries from Microsoft.Data.SqlClient is now really easy. And transitive dependencies are properly handled!

<ItemGroup>
  <ChiselPackage Include="Azure.Identity" />
  <ChiselPackage Include="Microsoft.IdentityModel.JsonWebTokens" />
  <ChiselPackage Include="Microsoft.IdentityModel.Protocols.OpenIdConnect" />
</ItemGroup>

<ItemGroup>
  <PackageReference Include="Chisel" Version="1.1.2" PrivateAssets="all" />
  <PackageReference Include="Microsoft.Data.SqlClient" Version="6.0.2" />
</ItemGroup>

I also updated the README in this repository to mention Chisel.

And thanks to the combination of dotnet/SqlClient#2577 and dotnet/SqlClient#2636, getting rid of Microsoft.Identity.Client doesn't require any special handling anymore. 🥳

@0xced 0xced closed this Jun 18, 2025
@virzak
Copy link
Author

virzak commented Jun 18, 2025

No worries at all.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants