-
Notifications
You must be signed in to change notification settings - Fork 148
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
LSP4J 0.22.0 #732
Comments
Because of fixes to handling 3rd party libs we need to do a release (or pre-release) of LSP4J so that we can see the effects in SimRel and resolve what Ed raised in https://www.eclipse.org/lists/cross-project-issues-dev/msg19709.html I can do a release now, but because of the current dep on a milestone of xtend I don't know if that is a good idea (#751) @cdietrich thoughts? |
The problem is I changed jobs on 1st so I won’t be really able to contribute much (specially on Xtext side ) |
OK - I guess we'll have to figure that out as a community! All the changes to LSP4J since 0.21.0 release have been third-party dep changes, so maybe best to simply do a 0.21.x release on 15 August with these changes backported: |
Hmmm... Is it even allowed to use yet-to-be-released content of other Eclipse projects in a release? IIRC, it was not allowed in the past... |
Xtext has to my knowledge no dep to the generator so we might be able to live with a broader range von Xtext side @szarnekow @tivervac any comments from your side |
It still is not - we are not going to release that content and #751 needs resolving to make sure that is handled. |
@cdietrich Is your question whether we use the |
Yes this is what I meant: use current lsp4j in xtext release, but allow newer release in manifest so that simrel can pull newer lsp4j |
I don't think LSP4J does semantic versioning (same as Xtext). This will make it a bit harder. I can see that it'd work to depend on LSP4J[current, current+2) instead of LSP4J[current, current+1) but this would also only reduce the probability of allowing inconsistent OSGI resolution. |
Can I release lsp4j 0.21.1 with guava dependency range increased to: |
This is what I mean explicitly: #759 |
Hey there, any idea when 0.22.0 is gonna land? |
I think we should schedule this to release, well in time for 2024-03 M3? That would mean around or before 13 Feb. cc: @cdietrich @mickaelistria @pisv for input on dates and coordination with your consumption of LSP4J. In particular, it would be nice if we all had 2024-03 using the same version of LSP4J. |
we wont have a new xtend release though. otherwise ok. |
Because of https://status.eclipsestatus.io/maintenance/315201 I may have trouble getting 0.22.0 done today. Builds are not running on Jenkins with errors like "forbidden: exceeded quota: jenkins-instance-quota" |
I'm part way through the release steps but I have to step away for a meeting. I'll resume later today. |
The release is now complete. The maven should update once their scripts run and I am now preparing the repo for 0.23.0 development (And can prepare for 0.22.1 if needed) |
This is the Release plan and TODO list for LSP4J release v0.22.0.
Steps for Release
Items at the beginning of development
s/0.21.0/0.22.0/g
,s/0.20.0/0.21.0/g
and review changes.) Ensure that-SNAPSHOT
is restored in the gradle/versions.gradle and releng/pom.xmlsh './releng/deploy-build.sh'
in releng/build.JenkinsfileItems in the days ahead of Release day:
japicmp
report and publish it as part of the build. Ensure that the API versions are incremented accurately based on the report. The reports are part of the build in japicmp-report and generated byreleng/runjapicmp.sh
Items on Release day:
-SNAPSHOT
from gradle/versions.gradle-SNAPSHOT
from releng/pom.xml entries in<dependencies>
section.sh './releng/deploy-build.sh'
in releng/build.Jenkinsfilev0.22.0
LSP4J_PUBLISH_LOCATION
->updates/releases/0.22.0
( <-- check version number)PROJECT
->lsp4j-multi-build/job/main
LSP4J_BUILD_NUMBER
-> the build that was just run aboveDRY_RUN
->false
v0.22.0
git tag -a v0.22.0 HEAD -m"LSP4J 0.22.0" && git push origin v0.22.0
-SNAPSHOT
need to be done right away.The text was updated successfully, but these errors were encountered: