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

Sign assemblies #7

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

Conversation

ermshiperete
Copy link

This PR creates strong-named versions of the assemblies. While this
doesn't have any benefits in regards to security it helps those projects
that (for whatever reason) are required to create strong-named assemblies
because all dependencies have to be strong-named as well.

Since signing an assembly doesn't help in any ways in regards to security,
it doesn't hurt if we add the key file to the repo.

This PR also adds the version and file version to the assemblies. It also
fixes the nuget package version for Sandwych.QuickGraph.Core.

This change creates strong-named versions of the assemblies. While this
doesn't have any benefits in regards to security it helps those projects
that (for whatever reason) are required to create strong-named assemblies
because all dependencies have to be strong-named as well.

Since signing an assembly doesn't help in any ways in regards to security,
it doesn't hurt if we add the key file to the repo.
This change adds the version and file version to the assemblies. It also
fixes the nuget package version for Sandwych.QuickGraph.Core.
@ermshiperete ermshiperete changed the title Signed assemblies Sign assemblies Jul 29, 2020
@oldrev
Copy link
Owner

oldrev commented Jul 31, 2020

Thanks for your contribution, I'll find some spare time to fix the CI problem(s).

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