Skip to content

Conversation

@soul2zimate
Copy link
Collaborator

@soul2zimate soul2zimate commented Jul 2, 2025

fix: clarify IntelliJ Ultimate Edition requirement for Golang projects.

downstream issue: https://issues.redhat.com/browse/TC-1980

fixes: #194

@soul2zimate soul2zimate self-assigned this Jul 2, 2025
@soul2zimate soul2zimate requested a review from ruromero July 2, 2025 01:50
@sonarqubecloud
Copy link

sonarqubecloud bot commented Jul 2, 2025

Copy link
Collaborator

@ruromero ruromero left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

First question is Why is this a requirement? isn't it just enough to have the binary available and watch go.mod files?
If it is required -> Can we let users know at runtime that this is a requirement for golang projects?
Maybe we can programmatically check if the Go plugin is installed, if not let the user know it is required.

@soul2zimate
Copy link
Collaborator Author

Hi @ruromero, this is a requirement since the support of GoLang analysis as DA plugin needs GoPlugin's help to analyze the language. E.g. https://github.com/redhat-developer/intellij-dependency-analytics/blob/main/src/main/java/org/jboss/tools/intellij/componentanalysis/golang/GoCAAnnotator.java#L14-L16, and the Go plugin is only compatible with IntelliJ IDEA Ultimate, GoLand, not IDEA Community.

This information was added to plugin release note when it supports Go for the first time, unfortunately, it misses in plugin description and project README.

@ruromero
Copy link
Collaborator

ruromero commented Jul 7, 2025

I have created this issue after our conversation #196

@soul2zimate
Copy link
Collaborator Author

Hi @ruromero, I think we should merge this to clear the current requirement for Go analysis before #196 is addressed in future release. Do you agree?

Copy link
Collaborator

@ruromero ruromero left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes let's merge this for this release and remove the dependencies in the next one.
Thanks

@soul2zimate soul2zimate merged commit 615619b into redhat-developer:main Jul 17, 2025
9 checks passed
@soul2zimate soul2zimate deleted the issue194_go_IJ_Ultimate_main branch July 17, 2025 07:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Unclear IntelliJ Ultimate Edition requirement for Golang projects analysis

2 participants