-
Notifications
You must be signed in to change notification settings - Fork 2.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
Release v2.0.0 #16428
Comments
We'll start this process after PR #16402 is merged. |
Don't do this yet, please. We had tons of regressions, the last one you discovered yesterday, so let's wait a bit to see if anything new will be discovered. @sarahelsaig is also still working on upgrading Orchard Core Commerce (see OrchardCMS/OrchardCore.Commerce#454) and other solutions to the latest preview. Since the goalpost keeps moving with new issues discovered, this is far from straightforward. |
@Piedone yes no need to release this immediately. But as far as I know that @sarahelsaig has been working on this for a while now and found one bug only "which was fixed". So I assumed there was no additional bugs found. Either way, at some point we have to draw a line and just ship so we can allow wider audience to test the new release. |
Actually I've found 11 bugs while testing with OCC and the Lombiq modules. The latest was fixed only 3 weeks ago. Also earlier this week I have found a new |
The point though is, having OCC and a few other large Lombiq solutions upgraded, and their over 300 UI tests passing, is a strong evidence that we're ready for a release. Otherwise, we'll have subsequent patch releases every other day. This has been a really bumpy road, getting blocked repeatedly. Version 2 will be a very risky release, hence we had a huge amount of regressions reported. So, due to the lack of a comprehensive automated QA suite, this is our second-best bet to make sure it'll go well. We can prevent further moving the goalposts by not merging anything else than bugfixes though. |
@Piedone Ideally we want Lombiq projects converted and all tests pass. But, will this happen any time soon? We just don't want to hold the release indefinitely without having some sort of plan to release. Is there a target finish date for Lombiq transition?
@sarahelsaig beside that ISession issue, are there any other issue from the 11 you reported that still need to be addressed? |
I'll defer about the timing to Sári. However, this also depends on what changes go to OC in the meantime. |
Once we have timeline, we'll be able to determine what will go into 2.0 and what will have to wait. But we should be looking into releasing 2.0 before August 1st as a max date. 1.8.0 was released in January 2, 2024. So we are well over due for a release. |
We triaged this issue and set the milestone according to the priority we think is appropriate (see the docs on how we triage and prioritize issues). This indicates when the core team may start working on it. However, if you'd like to contribute, we'd warmly welcome you to do that anytime. See our guide on contributions here. |
@Piedone I think this line item you added to the to do list. Both |
Are there any drawbacks to gitub's nuget service? |
Do you mean GitHub Packages? It isn't free, unlike Cloudsmith. |
@Piedone I tried a long time ago without any cost |
I see, it's actually free for public packages: https://docs.github.com/en/billing/managing-billing-for-github-packages/about-billing-for-github-packages#about-billing-for-github-packages |
Yes it's, it might be suited for us, especially for nightly builds, the thing I suffered from at that time was referencing them, from another apps |
@Piedone I think the issue is that these files are ignored from In this PR, I removed the exclusion rule so that we can track changes to the |
I don't feel like moving to GitHub Packages BTW, it seems like a pretty equivalent service. |
@Piedone It looks like @sarahelsaig completed the migration of e-commerce project and everything is passing. Are there any other project that we should wait for before we resume the release of 2.0? |
Yeah, she's working on DotNest as well, which is a different kind of test with all the random people's tenants. Note though that every breaking change like #16493 necessitates another round of updates. I'd also be a lot more confident if we can go a week without a regression being reported. |
This update should not need any code change. Rebuilding the project should be all one will need. I am okay with waiting a week to waiting. But the question, how long will the DotNest upgrade takes? @sarahelsaig in there an rough ETA for DotNest? |
Great, then! |
Unless there are some new surprise breaking change, I will finish it in the next couple days. |
@MikeAlhayek Good news, we are done testing with DotNest. |
@sarahelsaig that is great news! Do you want to tackle #16585 ? |
I did it here: #16623. |
Nothing else remaining here before release, right? |
I need to update PO Extract tool as well Can we expect OC |
The plan is to release OC early next week. So feel free to update the translations. |
IMHO we could do it in the OC Harvest :) meanwhile, we can test the latest fixes and check the translations, and integrations. Also OC Commerce and DotNest could do a final check before releae |
We want to release it before so we can talk about it and about 3.0 during the harvest. |
I'm not sure in which session we can talk about the current and future releases |
Probably during Q/A or side chats. I also think it's part of the keynote we are mentioning 3.0 |
@agriffard can you please release translations package for 2.0.0? |
Can we hold for 1-2 days I need to bump PO Extractor update then @agriffard can release translation packages for |
@hishamco tomorrow by noon PST, we'll need to release 2.0. I hope you'll be done by then. Since the Harvest is next week, some people will be out and/or busy. So we can't wait much longer. |
Let us wait for @agriffard, if the current PO Extractor version is fine I will defer bumping the new version after |
I updated the translations last week: OrchardCMS/OrchardCore.Translations@0bb805e I checked this task: I am not doing anything more.
|
@sebastienros could you please handle this or shall one of us do it? |
@agriffard Is this the version we should be using then? https://cloudsmith.io/~orchardcore/packages/?q=version%3A1.0.1-beta-20060 Shouldn't we call this version 2.0.0-preview to match OC versions? |
I made an update with the latest changes.
I don't know where this version of OC.Translations in preview is set. |
@hishamco what are you asking here? Are we still waiting on you to make any changes to the |
What I mean is to publish the |
@hishamco every commit on that project generates a new translation package. I can release a production version tomorrow "if you finished everything you want to do". |
I am going to delay this until Monday/Tuesday. We have one report of a 2.0 bug that we are still working on and we'll give people time to test the last changes over the weekend. Any objections? |
Go ahead! |
I defer everything after |
@Piedone is it time to do this yet? |
It's still on 9.x so I don't think so. |
Prepare the project
Do some housekeeping on GitHub in the main repo.
Prepare the code and documentation
Update the source, so everything looks like on the new version.
release/<version name>
branch (e.g.,release/2.0.0
) frommain
or the previous release branch. This is necessary to target pull requests for the upcoming release.OrchardCore.Commons.props
file with<VersionSuffix></VersionSuffix>
such that preview build numbers are not injected in packages. Verify theVersionPrefix
tag matches the released version.src/OrchardCore/OrchardCore.Abstractions/Modules/Manifest/ManifestConstants.cs
.dotnetcli.host.json
andtemplate.json
files.mkdocs.yml
and remove it fromnot_in_nav
.Test the release
Make sure everything works all right.
OrchardCore.Samples
works.release/
are automatically published too). Test at least the following guides:Prepare and publish Orchard Core Translations
Update everything in the Translations project. Only do this once all the code changes are done since localized strings can change until then.
OrchardCore.Translations.All
package reference in the main repo's ./Dependencies.Packages.props file to refer to the new NuGet package.Publish the release
Do the harder parts of making the release public. This should come after everything above is done.
release/<version name>
with the version. Includev
in the name, e.g.v2.0.0
.release/<version name>
tomain
.release/<version name>-integration
branch and fix them there.For details on this version see the [release notes in the documentation](link here).
). Note that the docs will only be built once the branch is merged tomain
.After the release is done
OrchardCore.Docs
project. (e.g.,/releases/2.0.0.md
). Don't add it to the docs navigation and exclude it from validation undernot_in_nav
withmkdocs.yml
.OrchardCore.Commons.props
file with the next release number, and<VersionSuffix>preview</VersionSuffix>
such that preview builds use the new one.Publicize the release
Let the whole world know about our shiny new release. Savor this part! These steps will make the release public so only do them once everything else is ready.
The text was updated successfully, but these errors were encountered: