-
Notifications
You must be signed in to change notification settings - Fork 12
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
CDT LSP 1.1.0 #210
Comments
I don't know if we are going to do 1.1.0 next, but the master branch should be updated to 1.1.0 now (and may become 2.0.0 if we want to do a major version). If we want to do a 1.0.1 version we should do that from the cdt_lsp_1_0 branch. |
@ghentschke thanks for all the work on getting the release out. There is a minor error, the wrong commit was tagged as 1.1.0. It should have been commit de6e9a5 (the one built in https://ci.eclipse.org/cdt/job/cdt-lsp/job/cdt_lsp_1_1/2/) but it was the next one 5c2a608 (which is the version bump that was on main branch) I think to resolve you can just add a comment in the release page, or try to delete the tag and recreate it:
But the protection rules will probably prevent you doing that, which means raising a helpdesk ticket |
Thank you, @ghentschke and @jonahgraham for quickly preparing the roll-out. Unfortunately, checking the update sites, I found some additional issues.
It seems, the released version (commit de6e9a5) contains version 1.0.0 in the MANIFEST.MF files. |
That's correct. The plugin version has not been incremented. I know that's a little irritating.
That should be also OK, since the xml files pointing to @jonahgraham Should the bundle version be incremented as well when we are not performing any API baseline checks? |
Ok. Thank you for clarification, @ghentschke. I didn't expect that. :-) But thinking about plug-in dependencies, I think, the bundle version has to be incremented, since the build number is usually not used when describing plug-in dependencies, but plug-ins using the new API do need to specify a newer version than 1.0.0 in order to force Eclipse to update the cdt lsp plug-ins. I've expected the new bundle versions to be 1.1.0.x. |
That is indeed irritating. Could you give a hint why it wasn't incremented? As @travkin79 mentioned, version increment is important to consumers that want force the minimum version range to be used. |
I know that its not good practice. I simply forgot it. Unfortunately I forgot to tag the exported packages of the LSP plugin as provisional API as well. I will do that soon. That why we added this comment to the changelog. |
FWIW the "correct" bundle versions should have been 1.0.100.qualifier for any bundles modified since 1.0.0. The qualifiers set from jgit + compare and replace are coming in the future. |
This is the Release plan and TODO list for CDT LSP.
Steps for Release
Items at the beginning of development
feature.xml
versionpom.xml
versionItems in the days ahead of Release day:
1.0.0 (CDT LSP)
)Items on Release day:
git tag -a CDT_LSP_1_0_0 HEAD -m"CDT LSP 1.0.0" && git push origin CDT_LSP_1_0_0
The text was updated successfully, but these errors were encountered: