Removed unused repository paths from repositories.config #1860
Conversation
| <repository path="..\src\NUnitEngine\nunit.core.engine\packages.config" /> | ||
| <repository path="..\src\NUnitEngine\nunit.engine.tests\packages.config" /> | ||
| <repository path="..\src\NUnitEngine\nunit.engine\packages.config" /> | ||
| <repository path="..\src\NUnitEngine\nunit.v2.driver.tests\packages.config" /> | ||
| <repository path="..\src\NUnitFramework\testdata\packages.config" /> | ||
| <repository path="..\src\NUnitFramework\tests\packages.config" /> |
There was a problem hiding this comment.
Odd, is this supposed to be repeated, but with inverted / ? seems a bit odd to me never noticed this before.
There was a problem hiding this comment.
No, this file is supposed to be auto-generated, but I think we were trying to be smart after a Linux user complained the Windows \ didn't work and switched them over. I think they got auto-readded back the windows way. @zharro can you remove the duplicates with the Linux slashes? I think if we remove the Windows versions they will just end up getting re-added.
|
Yes, I think it should include the |
|
Looks like Linux build failed after removing Linux slashes. I'll try to figure out the reason. |
|
@zharro - The test error looks unrelated to your changes. They're occasionally a little flakey. I've restarted the build, hopefully they'll pass the second time, and we can merge! |
|
All passing now. Looks good, merging. Thanks. |
Fixes #1827: removed all but NUnitFramework paths.