dotnet / dotnet/machinelearning

Remove System.CodeDom reference in Microsoft.ML nuget package

Open
#6,006 1 comment 0 reactions 1 assignee Claimed by @svick View on GitHub
area-Infrastructure
Dominant language
C#
Stars
9.4k
Forks
2k
Avg merge
2d 20h
Merged PRs (30d)
11

Description

Today Microsoft.ML has a dependency on System.CodeDom, but I don't see anything actually requiring this reference:

https://github.com/dotnet/machinelearning/blob/ea647f43246989334bad69f35b5d60d15171c57e/src/Microsoft.ML/Microsoft.ML.csproj#L44

The only references to `System.CodeDom` in the code is to use IndentedTextWriter, which doesn't require a reference to System.CodeDom.

We should remove this reference from our core NuGet package.

cc @ericstj @michaelgsharp

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.