-
Notifications
You must be signed in to change notification settings - Fork 689
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
Fixes #3178. v1 - Library has no strong-name. #3191
Conversation
Add strong-name signing as requested for issue gui-cs#3178
Added strong name to UnitTests project too.
I suppose that NStack library should be strong-named too. |
Created PR #91 for NStack. I tried to build again this version of Terminal.Gui with it and strong-name warning has now disappeared. |
Yeah, that's worded a bit unfairly. Please, at least, do this:
|
I don't think so and the tests pass. But you have to wait for @tig's decision. |
Do I need to do anything other than merge this for it to work for publishing to Nuget? I was expecting to have to generate a private key that we'd need to store in a github secret or something. |
It is not necessary to generate a password for the .snk file for strong name purposes, so you just need to merge it. Thank you, @tig. |
Ok, NStack v1.1.1 is published with an SN. Now for Terminal.Gui... |
v1.15.1 is published. Please verify that I didn't screw something up ;-) |
Add strong-name signing.
Fixes
Terminal.Gui
v1 #3178Proposed Changes/Todos
Pull Request checklist:
CTRL-K-D
to automatically reformat your files before committing.dotnet test
before commit///
style comments)