-
Notifications
You must be signed in to change notification settings - Fork 602
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
Integrated set of Liberty tools for developers downloadable from IntelliJ IDEA's marketplace #14957
Comments
POC Forum UFO feedback: |
Given the feedback from the POC Forum, the Serviceability and Accessibility Compliance slides have been updated. |
UFO Socialization Feedback: Feature Design: Which Projects to List on Dashboard (Slide 22): Discussion regarding improving the maven flow using the fully qualified maven coordinates, for the case where it's not defined in the pom.xml (but a server.xml exists). Feature Design: Manually adding projects (Slide 23): Clarification that this is configured per project (and persisted to workspace) Feature Design: LSP Support (Slide 28) - Show where LSP comes from and show how LSP connects to IDE and what each provides. Rework the diagram, explain what is being picked up from intellij-quarkus (and the potential for making it a common component) Feature Design: MicroProfile & Jakarta EE LS Features (Slide 31): Explain JDT LS and how it will be augmented. Communication (Slide 34): Add IntelliJ Plugin Marketplace Performance (Slide 46): How are things like project specific Java SE version set (for use during dev mode)? Are we providing this, or is it already being provided by existing plugin functionality? |
Updated the UFO (https://ibm.box.com/s/9nkbfn2knpvy51yu3rjrvyjhibfg2phb) to incorporate the feedback from the design review. Notable key changes:
|
@NottyCode please review the updated design. Thank you |
When reviewing one of the things I look to ensure is that feedback from socialization is addressed. In this case there are a few items which I cannot tell if they have been addressed. Note there might be a good reason for something to not result in a UFO update, but in that case a comment to indicate why is a good thing for me when reviewing. The following things appear not yet addressed:
I included this because I'm not sure how the current version of the slide addresses the feedback .
I included this because I'm not sure how the current version of the slide addresses the feedback .
it isn't clear this is addressed at all |
@NottyCode Here is a more detailed breakdown of how each socialization comment has been addressed in the design document. I've also made a few minor updates to the design to make these items clearer. https://ibm.box.com/s/9nkbfn2knpvy51yu3rjrvyjhibfg2phb
Updated "As-Is/To-Be: Running dev mode" (slides 10 & 11) to clarify that users will have to configure the Liberty Maven/Gradle plugin in the build file if it is not already present. Issue OpenLiberty/liberty-tools-intellij#113 was opened for an enhancement to the plugin to help users configure the Liberty build plugins (post GA).
Updated "Feature Design: Start vs Start..." (slide 19) and "Feature Design: Dev mode support" (slide 20) to indicate that the fully qualified Maven coordinates will run for Maven commands. This will cover the case where the Liberty Maven Plugin is not defined in the pom.xml but a server.xml file exists. There is not an equivalent story for Gradle yet, will be addressed by the future enhancement to help users configure the Liberty build plugins (OpenLiberty/liberty-tools-intellij#113).
Updated "Feature Design: Manually adding projects" (slide 25) to clarify that this is configured per project and persists between IntelliJ sessions.
Updated "Feature Design: LSP Support" (slide 30) to contain a more detailed component diagram. The diagram now indicates that the LSP for IntelliJ implementation is to be reused from intellij-quarkus, and added a note that the long term goal will be a common LSP for IntelliJ component that both projects can consume.
Updated "Feature Design: MicroProfile & Jakarta EE LS Features" (slide 33) to clarify what the JDT LS is and why it is needed in Eclipse LSP4MP & LSP4Jakarta. Clarified that we will re-implement the JDT LS components using the IntelliJ Java PSI API to deliver the same Java language server capabilities in IntelliJ.
Updated Communication (slide 36) to call out the IntelliJ Plugin Marketplace.
Updated "Feature Design: How to configure Maven/Gradle/Java" (slide 26) to clarify that the IntelliJ terminal honours the Maven/Gradle/Java set for the local machine. This setting determines which Java SE version the terminal runs Liberty dev mode with. Users can override the environment for the terminal using IntelliJ "Preferences" -> "Tools" -> "Terminal" settings. Users can also set the Java compiler version for Maven/Gradle projects in their build file. We do not offer additional configuration other than what is already provided by the IntelliJ terminal preferences. |
Thank you, Kathryn, Alasdair! |
Enable developers that use IntelliJ IDEA as their IDE to pull down and install from its marketplace an integrated set of tools to code/build/test/debug their Java applications on top of Liberty easily using APIs like Jakarta EE and MicroProfile.
Solution involves creating an IntelliJ plugin for Liberty that enables developers to easily create and develop cloud-native Java applications with Liberty in IntelliJ IDEA.
Plugin should include:
When ready, add links to the Upcoming Feature Overview document as well as Feature Test Summary and blog post issues:
List of Steps to complete or get approvals / sign-offs for Onboarding to the Liberty release (GM date)
Instructions:
Design
Before Development Starts or 8 weeks before Onboarding
Beta
If your feature, or portions of it, are going to be included in a beta
Before Onboarding the beta
kind=beta
,ibm:beta
,ProductInfo.getBetaEdition()
)1 week before beta GA
Legal
3 weeks before Onboarding
Translation
3 weeks before Onboarding
Feature Complete
2 weeks before Onboarding
Focal Point Approvals
2 to 1 week before Onboarding
You MUST have the Design Approved or No Design Approved label before requesting focal point approvals.
All features (both "Design Approved" and "No Design Approved")
"Design Approved" features
Ready for GA
1 week before Onboarding
1 week before GA
Other deliverbles
The text was updated successfully, but these errors were encountered: