Skip to content

Update testProject nuget packages and targetframework#122

Open
eoehen wants to merge 2 commits into
magic-chunks:masterfrom
eoehen:feature/Update-TestProject-Nugets
Open

Update testProject nuget packages and targetframework#122
eoehen wants to merge 2 commits into
magic-chunks:masterfrom
eoehen:feature/Update-TestProject-Nugets

Conversation

@eoehen

@eoehen eoehen commented Jan 18, 2021

Copy link
Copy Markdown

Update nuget packages of the testproject:

  • Microsoft.NET.Test.Sdk from 15.5.0 to 16.8.3
  • xUnit from 2.3.1 to 2.4.1

TargetFramework from netcoreapp2.0 to netcoreapp 2.2
DotNetCliToolReference dotnet-xunit removed

@gep13

gep13 commented Jan 18, 2021

Copy link
Copy Markdown
Member

@eoehen thank you for taking the time to raise this PR. Was there a specific reason that you were looking to have these changes included?

We are currently working on the 2.1.0 release of Magic Chunks, so this change likely won't be included in that release, but we may consider it after that release is completed.

@pascalberger pascalberger left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@eoehen Thanks for your contribution. There's one thing which additionally needs to be updated manually.

Comment thread src/MagicChunks.Tests/MagicChunks.Tests.csproj Outdated
@eoehen eoehen force-pushed the feature/Update-TestProject-Nugets branch from 9a0ebd7 to 70d3f13 Compare January 18, 2021 20:24
@eoehen

eoehen commented Jan 18, 2021

Copy link
Copy Markdown
Author

@gep13 Sorry for the misunderstanding. I ask @pascalberger first, if I can create a PR for updating this packages. You can use it for a later version.

Comment thread src/MagicChunks.Tests/MagicChunks.Tests.csproj Outdated
@eoehen eoehen force-pushed the feature/Update-TestProject-Nugets branch from 70d3f13 to f4db813 Compare January 18, 2021 21:22

@pascalberger pascalberger left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@pascalberger pascalberger left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@eoehen

eoehen commented Jan 18, 2021

Copy link
Copy Markdown
Author

@pascalberger Build is fixed.

@pascalberger pascalberger left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

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.

3 participants