Skip to content

dotnet-core: Bumping versions, cleaning up test-project#77

Open
mnsc wants to merge 3 commits into
swkBerlin:masterfrom
mnsc:master
Open

dotnet-core: Bumping versions, cleaning up test-project#77
mnsc wants to merge 3 commits into
swkBerlin:masterfrom
mnsc:master

Conversation

@mnsc
Copy link
Copy Markdown

@mnsc mnsc commented Oct 27, 2020

dotnet-core-version wasn't working, bumped all versions to .net 5 (tested on .net5 rc2) and now it works.

While doing this I cleaned up the csproj by removing the project and recreating it with the default xunit test project template (VS2019 preview).

What got removed in the cleaning:

"Given that there's no point in packing test projects..."
dotnet/templating#376

"Note that DotNetCliToolReference is now deprecated in favor of .NET Core Local Tools."
https://docs.microsoft.com/en-us/dotnet/core/tools/csproj#dotnetclitoolreference

Magic guid that says "test project"
microsoft/vstest#472

mnsc added 2 commits October 27, 2020 15:55
TargetFramework to .net 5.0
All dependencies to latest as of now.
Cleaned up the csproj by removing the project and recreating it with the default xunit test project template (VS2019 preview)
@mklose
Copy link
Copy Markdown
Member

mklose commented Oct 27, 2020

Hello @mnsc thank you ❤️ for contributing !

Would you be keen to setup CI (see README) for it too?

@mnsc
Copy link
Copy Markdown
Author

mnsc commented Oct 28, 2020

I'm interested in learning more about github actions so I can give it a shot! It's kinda weird setup though to expect one test to fail but maybe I can get some inspiration from the other workflows.

@mklose
Copy link
Copy Markdown
Member

mklose commented Nov 10, 2020

Hi @mnsc

It's kinda weird setup though to expect one test to fail

the idea behind it is to make it super easy for people just to start (even if they don't know the particular language or/and testing framework), since running the tests will point you to the next step.

Remove workflows (to be reverted)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants