Help IDEs resolve gen/include immediately#118
Open
van800 wants to merge 1 commit intogodotengine:mainfrom
Open
Help IDEs resolve gen/include immediately#118van800 wants to merge 1 commit intogodotengine:mainfrom
van800 wants to merge 1 commit intogodotengine:mainfrom
Conversation
…can see generated headers immediately after generate_bindings target produces them
Contributor
Author
|
@enetheru, what do you think? |
Contributor
|
is there changes missing? how does creating that directory effect anything? |
Contributor
Author
|
I suppose, if folder does not exist, IDE can't watch it for further changes. I haven't dug too deep, but I checked the behavior is consistent with Clion, Rider and VS. |
Contributor
well if it helps i'm all for it. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Create the godot-cpp/gen/include directory at configure time so IDEs can see generated headers immediately after generate_bindings target produces them.
The problematic use-case, which I have fixed for Rider, CLion and VS:
example_class.cppobserve red code
generate_bindingstargetstill observe red code, while expected result would be to see all resolved.
Notes: