-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
NLog 5.0 Ready for delivery #4802
Conversation
e34eed5
to
1ffd024
Compare
413cd53
to
59739c2
Compare
@304NotModified Anything more to include, before merging ? Would prefer to have NLog/NLog.github.io#181 merged together with the NLog 5.0-release And to have NLog/NLog.github.io#183 merged, so I can continue the work with actually adding the new NLog 5.0-packages to the list (Ex. NLog.Database-nuget-package) |
I think we have to update the xsd online? |
I'm fine with starting out with NLog schema nuget-package only. Have no opinion about having NLog5.xsd or just overwriting NLog.xsd
|
e6faf1d
to
75078a4
Compare
6a9854c
to
23ad02d
Compare
33317ba
to
ae47eff
Compare
@304NotModified I guess we are super close to perform the official release ? (Only have #4887 in queue) |
guess only some docs work? https://github.com/NLog/NLog/issues?q=is%3Aopen+is%3Aissue+milestone%3A%225.0+%28new%29%22 |
I was thinking the release-build-first then followed by release-news-post, and finally look at generating the source-code-docs (Since NLog wiki is already updated). |
1b08a4e
to
2dc678c
Compare
@304NotModified Any objections on pushing the merge-button, and building NLog 5.0 RTM ? |
01a5786
to
0fddce6
Compare
Will try to double check today/this weekend! Im a bit behind all the notifications |
d0fc13b
to
256da80
Compare
Will be tomorrow (Monday) evening. I have blocked some time for it. |
Kudos, SonarCloud Quality Gate passed! |
@snakefoot here we go! |
oops broke the link - my bad! |
The release looks great, and thank you for fixing the broken link that I forgot. Really like the ReadMe you created for the NLog nuget-package. Good idea with using the headlines. |
thx. No I broke the link and fixed it with a script on the 404 site 😄 |
Removed RC-suffix
See also: https://github.com/snakefoot/NLog/blob/nlog5-ready/CHANGELOG.md
Also merge release-posts: NLog/NLog.github.io#181 + NLog/NLog.github.io#188