search

Leagify / prospect-scraper-dt2021

Scraping draft prospects for fun and charts.
1 stars 9 forks source link

Test project not included in the main solution #27

Closed robsmithw closed 4 years ago

robsmithw commented 4 years ago

Any reason the prospectScraperTest.csproj (test project) is not included in the prospectScraper.sln (main solution)? I was making a change on another issue and noticed this, if the test project was part of the main solution, any contributors using Visual Studio to develop would see the test project regardless of what they're making a change to and would prompt contributors to be more likely to run the test before commiting changes (I would hope 😄).

github-actions[bot] commented 4 years ago

Thank you for putting in your first issue!

zo0o0ot commented 4 years ago

I don't have any problems with getting it added. I've been doing most of my development in GitPod, which uses the csproj files, not the sln, which is probably why I missed that.

zo0o0ot commented 4 years ago

This was actually handled in #26