feat: Add reference and rename providers - #12
Open
depau wants to merge 2 commits into
Open
Conversation
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
1 participant
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.
Disclaimer: I used AI assistance to develop this functionality. A quick look at your profile seems to suggest you're ok with that, but feel free to close if not interested.
This change introduces support for Ctrl+Click / Go to definition navigation for
gcode_macroanddelayed_gcode, as well as support for renaming.The providers parse config includes and build an include tree, then detect all macro blocks in the tree and provides references to the detected blocks in all gcode templates.
EDIT: I also added support for macro references in built-in macros that reference other macros:
SET_GCODE_VARIABLE MACRO=<macro>,TUNING_TOWER COMMAND=<macro>, andUPDATE_DELAYED_GCODE ID=<delayed_gcode>.