getgauge / gauge-csharp

Csharp runner for Gauge
https://gauge.org
GNU General Public License v3.0
33 stars 21 forks source link

Nuget package DLL's in gauge_bin folder #157

Closed manavghaigithub closed 5 years ago

manavghaigithub commented 5 years ago

Expected behavior

Gauge should remove DLL's from gauge_bin folder once the package that was added through NuGet is removed from Gauge .Net Core project

Actual behavior

Gauge is not removing DLL's from gauge_bin folder once the package that was added through NuGet is removed from Gauge .Net Core project

Steps to reproduce

  1. Create a new .Net Core project through Gauge
  2. Create any sample/test Step Implementation
  3. Add Nuget Package through Manage Nuget in Visual Studio. For e.g. add Selenium.WebDriver.
  4. When you remove this package from Manage Package, the associated Selenium DLL's does not get removed from gauge_bin folder. They should get removed because i dont need them anymore.

Gauge version

Gauge version: 1.0.2 Commit Hash: b4288bb

Plugins

csharp (0.10.5) dotnet (0.1.0) html-report (4.0.6) screenshot (0.0.1)

BugDiver commented 5 years ago

@manavghaigithub The gauge_bin folder is used to build the project, which copies the required DLL's from nuget cache to gauge_bin. It happened at runtime. So when you remove the package from Manage Package gauge would know about it. But It should be removed when you run the tests next time or restart the VS.

We currently do not support .Net core for visual studio. Please refer https://github.com/getgauge/gauge/issues/1288#issuecomment-453373955

manavghaigithub commented 5 years ago

Hey Vijay - I will try to remove package from terminal in VS Code and then will restart VS Code to see whether it is happening or not.

Thanks, Manav Ghai


From: Vinay Shankar Shukla notifications@github.com Sent: Wednesday, February 6, 2019 4:17:08 AM To: getgauge/gauge-csharp Cc: Ghai, Manav (Cognizant); Mention Subject: Re: [getgauge/gauge-csharp] Nuget package DLL's in gauge_bin folder (#157)

@manavghaigithubhttps://github.com/manavghaigithub The gauge_bin folder is used to build the project, which copies the required DLL's from nuget cache to gauge_bin. It happened at runtime. So when you remove the package from Manage Package gauge would know about it. But It should be removed when you run the tests next time or restart the VS.

We currently do not support .Net core for visual studio. Please refer getgauge/gauge#1288 (comment)https://github.com/getgauge/gauge/issues/1288#issuecomment-453373955

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHubhttps://github.com/getgauge/gauge-csharp/issues/157#issuecomment-460952068, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AsNVIGpo4p2PZ4jBMI0Ev_fevcqTZrJJks5vKp2UgaJpZM4ahqhb.

BugDiver commented 5 years ago

@manavghaigithub Closing it for now since .Net core is not supported on Visual Studio.