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.
Spent some time messing around with completions, curious to get your thoughts.
Note: I don't feel this is actually ready for merging yet, just looking to get some eyes on it
The basic approach is to add a file alongside Bakefile that will store the completion functions. When completion is called it will source that file then check if a function exists matching the naming convention
If so, that function will be used to provide completions (via white-space delimited stdout)
As I'm writing this, I'm realizing I could just move all completion responsibility to that function and have it simply provide completions via
COMPREPLYIf no function is found then we fallback on default completion.
Key drawbacks that I see:
Bakefile.completions) feels arbitrary as the completion functions must just exist in the shell.