Skip to content
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

Small fixes after upgrade to .NET 5.0 SDK #1800

Merged
merged 3 commits into from
Nov 17, 2020

Conversation

thomaslevesque
Copy link
Member

  • a new AssemblyMetadata attribute for RepositoryUrl is now added to assemblies
  • fixed a new nullability warning

There are still warnings NETSDK1138 warnings due to netcoreapp1.0 and netcoreapp3.0 being out of support, but at least we should have a working build again

@blairconrad
Copy link
Member

Thanks, @thomaslevesque. I generally like the changes, but Ubuntu doesn't.

@thomaslevesque
Copy link
Member Author

thomaslevesque commented Nov 17, 2020

but Ubuntu doesn't.

Oh. I can build without issue on WSL 2.
I suspect this is because AppVeyor still has a preview version of the .NET 5 SDK... Let's give it some time, it will probably be updated soon.

@thomaslevesque
Copy link
Member Author

@thomaslevesque
Copy link
Member Author

It's fixed!

Copy link
Member

@blairconrad blairconrad left a comment

Choose a reason for hiding this comment

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

👍
Thanks!

@blairconrad blairconrad merged commit 853069c into FakeItEasy:master Nov 17, 2020
@blairconrad blairconrad added this to the vNext milestone Nov 17, 2020
@thomaslevesque thomaslevesque deleted the net50sdk-fixes branch November 18, 2020 07:38
@thomaslevesque
Copy link
Member Author

Thanks for the merge. I have a pretty big one incoming after that, to address #1798.

@afakebot
Copy link

This change has been released as part of FakeItEasy 7.0.0-beta.1.

@blairconrad
Copy link
Member

This change has also been released as part of FakeItEasy 7.0.0.

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

Successfully merging this pull request may close these issues.

None yet

3 participants