-
Notifications
You must be signed in to change notification settings - Fork 179
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
Release 6.0.0-beta.1 #1671
Comments
#1690 is nearly complete, and this was the last issue that we'd listed as a must-have for the 6.0.0 release. Once it's merged, I'd like to issue a beta.1 release, if that's okay. |
Works for me.
I'm not very worried about NuGet. Octokit, I don't know, but I guess that if the code still compiles, the behavior probably won't have changed. Is there a benefit to upgrading, or is it just for the sake of upgrading? |
Yeah. Which is part of the reason I hadn't pushed it yet. I'm content to leave them as they are until we run into trouble. |
I don't mind either way. If you feel like upgrading, go ahead |
Nah, it's good. My stress level rises every time we jiggle the deploy tool. |
FYI I tried out the 6.0.0 beta, and found no regressions, tests, continued to run pretty fast, but .... our overall build time went up noticably! My hunch is that this is most likely to be a size-of-files, times number of times we have to copy/load/crossref them thing related to Castle.Core packaging. |
Glad there were no regressions, @TimLovellSmith. I'm a little surprised about the build time changes. Which TFM are you using? |
broken by ReturnValue not set ReturnValue not set #1696, released in 5.4.1.
SetReturnValue
is no longer found byFakeObjectCall
.tools\prepare_release.cmd next x.y.z vNext
to:The text was updated successfully, but these errors were encountered: