Skip to content

Add compiledb, help and custom.py support#5

Merged
paddy-exe merged 1 commit intogodotengine:mainfrom
adamscott:add-compiledb-and-custom-support
Oct 15, 2023
Merged

Add compiledb, help and custom.py support#5
paddy-exe merged 1 commit intogodotengine:mainfrom
adamscott:add-compiledb-and-custom-support

Conversation

@adamscott
Copy link
Member

@adamscott adamscott commented Aug 17, 2023

As the title says, it adds compiledb and custom.py support.

This is needed for new users for the process to be as smooth as possible. This includes the generation of compile_commands.json to enable autocompletion while creating C++ code.

Depends on godotengine/godot-cpp#1220

Edit: Added help support too

@paddy-exe paddy-exe added the enhancement New feature or request label Aug 25, 2023
@adamscott adamscott force-pushed the add-compiledb-and-custom-support branch from 7edcdfd to 941790b Compare September 7, 2023 17:44
@adamscott adamscott force-pushed the add-compiledb-and-custom-support branch from 941790b to 43d3b11 Compare September 7, 2023 18:01
@adamscott adamscott changed the title Add compiledb and custom.py support Add compiledb, help and custom.py support Sep 7, 2023
@adamscott
Copy link
Member Author

@paddy-exe godotengine/godot-cpp#1220 has been merged!

@paddy-exe
Copy link
Collaborator

@paddy-exe godotengine/godot-cpp#1220 has been merged!

Awesome👍🏻 but it should be backported first to merge it

Copy link
Collaborator

@paddy-exe paddy-exe left a comment

Choose a reason for hiding this comment

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

LGTM and the godot-cpp PR has been cherry-picked

@paddy-exe paddy-exe merged commit 055ab32 into godotengine:main Oct 15, 2023
@0dminnimda
Copy link

0dminnimda commented Nov 1, 2023

This (generation of compile_commands.json) does not work even with #1220. The resulting compile_commands.json is empty

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants