New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add NUnit Itemgroup to csproj #1282

Merged
merged 1 commit into from Dec 22, 2017

Conversation

2 participants
@Aergwyn
Member

Aergwyn commented Dec 22, 2017

Same thing as in ppy/osu#1673.
It gets included and marked as a change every time you run tests on VS which is pretty annoying.

@smoogipoo

This comment has been minimized.

Show comment
Hide comment
@smoogipoo

smoogipoo Dec 22, 2017

Contributor

osu.Framework doesn't include tests though? nvm didn't read rest of comment.

Contributor

smoogipoo commented Dec 22, 2017

osu.Framework doesn't include tests though? nvm didn't read rest of comment.

@smoogipoo smoogipoo merged commit 08f85f9 into ppy:master Dec 22, 2017

1 check passed

continuous-integration/appveyor/pr AppVeyor build succeeded
Details

@Aergwyn Aergwyn deleted the Aergwyn:add-nunit-include branch Dec 22, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment