Skip to content
This repository has been archived by the owner on Feb 23, 2021. It is now read-only.

Remove unnecessary package references #175

Closed
pranavkm opened this issue Mar 28, 2017 · 4 comments
Closed

Remove unnecessary package references #175

pranavkm opened this issue Mar 28, 2017 · 4 comments
Assignees

Comments

@pranavkm
Copy link
Contributor

We should do a scan and figure out if there are any package references we don't actually need in our projects. For instance, aspnet/Logging#585

@pranavkm
Copy link
Contributor Author

cc @davidfowl

@Eilon Eilon added this to the 2.0.0-preview1 milestone Apr 7, 2017
@Eilon
Copy link
Member

Eilon commented Apr 7, 2017

In theory the work is this:

  1. Find all csproj files in all src folders in all repos
  2. Delete all "System.*" package references
  3. If it compiles, send a PR and get it merged
  4. If it doesn't compile, figure out what's needed and go to step (3)

@pakrym
Copy link
Contributor

pakrym commented Apr 7, 2017

I was thinking about checking which package dll's got optimized out from assembly references by compiler.

@Eilon
Copy link
Member

Eilon commented Apr 9, 2017

@pakrym that could certainly work as well.

@Eilon Eilon modified the milestones: 2.0.0-preview2, 2.0.0-preview1 Apr 21, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants