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

Modernize project #136

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Modernize project #136

wants to merge 3 commits into from

Conversation

0xced
Copy link

@0xced 0xced commented Nov 21, 2024

  • Target .NET Standard 2.0, as recommended by Microsoft

    Targeting .NET Standard prior to 2.0 is no longer recommended. See https://aka.ms/dotnet/dotnet-standard-guidance for more details.

  • Depend on Hangfire.Core 1.7.0 instead of Hangfire.Core 1.6.0

  • Upgrade Newtonsoft.Json to version 13.0.3 in order to fix vulnerability

    NU1903 : Warning As Error: Package 'Newtonsoft.Json' 11.0.1 has a known high severity vulnerability, GHSA-5crp-9r3c-p9vr

  • Fix HTML rendering (properly escape %, < and >)

  • Don't use obsolete methods anymore

  • Update all test dependencies

  • Fix all xUnit and Moq warnings

  • Remove AssemblyInfo.cs files

* Target .NET Standard 2.0, as recommended by Microsoft
  > Targeting .NET Standard prior to 2.0 is no longer recommended. See https://aka.ms/dotnet/dotnet-standard-guidance for more details.
  
* Depend on Hangfire.Core 1.7.0 instead of Hangfire.Core 1.6.0

* Upgrade Newtonsoft.Json to version 13.0.3 in order to fix vulnerability
  > NU1903 : Warning As Error: Package 'Newtonsoft.Json' 11.0.1 has a known high severity vulnerability, GHSA-5crp-9r3c-p9vr

* Fix HTML rendering (properly escape %, < and >)

* Don't use obsolete methods anymore

* Update all test dependencies

* Fix all xUnit and Moq warnings

* Remove AssemblyInfo.cs files
* Use PackageIcon instead of PackageIconUrl
* Use PackageLicenseExpression instead of PackageLicenseUrl
* Embed the README file in the NuGet package
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.

1 participant