-
Notifications
You must be signed in to change notification settings - Fork 144
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
Nuget Publishing #696
Comments
This sounds good 👍 |
Short Update:
Due to my match-v5 implementation the tests are outdated and throw errors, therefore I could not include them. |
Implementation finished. please check the markdown file and add nuget api key to secrets. |
Implementation done and merged. |
yes I'll try on Monday I think 👍 |
Want to reask about trying setting a new Release. :) |
oops sorry, I'm on holidays starting next week. I should have more time then 👍 |
@xXLAOKOONXx I switched to https://github.com/MingweiSamuel/Camille |
As you can see in #706 the installation of riotsharp is a barrier for using the framework. If you have any questions or my documentation is not descriptive enough hit me up. |
Currently the publishing of nuget packages is a manual task reducing the agility and user experience of this wrapper.
"nuget publishing is currenly broken and we'd need someone to take it over." - BenFradet (#684 (comment))
Does this mean someone needs to implement the pipeline or take over the complete process or project?
I experimented a bit with nuget and github actions to create a pipeline that on tag/release creation is able to build and publish a nuget package. If this sounds good, I can write an action for RiotSharp. (I would need a nuget key for nuget publishing in the project secrets for the pipe to work, I can give more detailed explaination if this is the way to go.)
The text was updated successfully, but these errors were encountered: