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

Updated to complete using part of the target name and absolute paths. #105

Merged
merged 2 commits into from
Sep 8, 2022

Conversation

nielshaandbaek
Copy link
Contributor

The purpose of this PR is to make the completion functionality a bit smarter, and able to complete build targets based only on part of the name. If you have build targets such as:

  //exp-compute/hw/ipp/hdl/IppSynth
  //exp-compute/hw/ipp/hdl/Lint
  //exp-compute/hw/ipp/sim/Simulation

With this work as a user one can do e.g.

$ dbt build ipp

and hit tab which will replace your command line with

$ dbt build //exp-compute/hw/ipp/

and show the options of the three actual targets.

jfrohnhofen
jfrohnhofen previously approved these changes Jun 24, 2022
@nielshaandbaek nielshaandbaek force-pushed the nh/improving_completion branch from c97334c to 9cb30d5 Compare August 5, 2022 20:13
@nielshaandbaek nielshaandbaek merged commit 4b11115 into master Sep 8, 2022
@nielshaandbaek nielshaandbaek deleted the nh/improving_completion branch September 8, 2022 17:38
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

Successfully merging this pull request may close these issues.

2 participants