Skip to content
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

bazelbuild/examples indexing problem #14

Closed
tpasternak opened this issue Nov 5, 2020 · 3 comments
Closed

bazelbuild/examples indexing problem #14

tpasternak opened this issue Nov 5, 2020 · 3 comments

Comments

@tpasternak
Copy link
Contributor

Scenario:

  1. Import java-tutorial from https://github.com/bazelbuild/examples
  2. Open Greeting.java

Observed:
Greeting.java is not correctly indexed - package name is marked as incorrect

@SocksDevil
Copy link
Contributor

Thanks for noticing this! It would seem that this problem also affects our current development build of the server, which uses the same pathing.

Probably this should be looked into. Our heuristic for calculating roots seems to be correct, but the behavior is not as expected. I believe that the problem might be associated with the the value baseDirectory on each target, since it is pointing directly towards the folder where the source files are. My assumption is this since IntelliJ suggestion to move to the correct folder creates the whole package structure within that folder.

@jastice did anything change on your side for this?

@jastice
Copy link
Member

jastice commented Dec 8, 2020

Nothing really changed in the BSP support. Maybe there's some interaction with the enhanced package prefixes in 2020.3? I haven't looked into this yet.

@SocksDevil
Copy link
Contributor

Just tried to importing this through 2020.2 and found that this did not happen there. Going to close this and open an issue on youtrack about it. You can find it here

jastice pushed a commit that referenced this issue Oct 22, 2023
* needs more love after andrzej's pr

* import mechanism improvements (#14)

Co-authored-by: Marcin Abramowicz <marcin.abramowicz@jetbrains.com>

* id does something xd

Co-authored-by: Andrzej Głuszak <andrzej.gluszak@jetbrains.com>
jastice pushed a commit that referenced this issue Oct 22, 2023
Co-authored-by: azdrojowa123 <aleksandra.zdrojowa@jetbrains.com>
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

No branches or pull requests

3 participants