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.

Sign up for GitHub

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

Jump to bottom

feat: bash autocompletions #72

Merged
merged 32 commits into from
Oct 15, 2024
Merged

feat: bash autocompletions #72

merged 32 commits into from
Oct 15, 2024

Conversation

apskhem
Copy link
Contributor

@apskhem apskhem commented Oct 15, 2024 •
edited
Loading

  • Added command autocompletions for cat-forge, at this moment, it can list all subcommands.

Closes #66

In order to use autocompletions, you need to register the forge command first.

complete -C <abs-path-to-forge-bin> forge

And then you can use autocompletion

forge <TAB>

It should give the list of all subcommands.

apskhem and others added 24 commits October 3, 2024 17:36
@apskhem
feat: initial command registration
88f00ca
@apskhem
feat: extractor function
f0bff4f
@apskhem
feat: complete parsing
bd7ffe3
@apskhem
feat: id formatter
4dd7024
@apskhem
feat: process cmd
4a9e746
@apskhem
feat: executable
7319379
@apskhem
Merge branch 'master' into feat/initial-devx
983dd76
@apskhem
feat: stream output
1ea1931
@apskhem
chore: msg
2f2b52a
@apskhem
Merge branch 'master' into feat/initial-devx
add3272
@apskhem
chore: sync main
65cf45b
@apskhem
feat: check available command
118c7e0
@apskhem
chore: move test devx to testdata
8448731
@apskhem
refactor: move utils
9999874
@apskhem
refactor: restructure
73e5ea5
@apskhem
chore: remove comments
2b3322c
@apskhem
refactor: using executor
75f0b52
@apskhem
refactor: command executor matching
d9d9b2b
@apskhem
test: for devx
8de6e57
@apskhem
Merge pull request #63 from input-output-hk/feat/initial-devx
e88faa3 
feat: add initial DevX
@apskhem
feat: install kongplete
d4dc550
@apskhem
feat: root completion installation
6929997
@apskhem
refactor: main.go
7c85c74
@apskhem
feat: kong for cmds
3bd8e95
@apskhem apskhem added the enhancement New feature or request label Oct 15, 2024
@apskhem apskhem self-assigned this Oct 15, 2024
@apskhem apskhem linked an issue Oct 15, 2024 that may be closed by this pull request
Add autocompletion when looking up commands from the markdown definition #66
Closed
@apskhem apskhem marked this pull request as draft October 15, 2024 08:24
@apskhem apskhem force-pushed the feat/devx-autocomplete branch 2 times, most recently from 0b4c0fa to 2b98a4d Compare October 15, 2024 08:40
@apskhem apskhem marked this pull request as ready for review October 15, 2024 09:21
@apskhem apskhem marked this pull request as draft October 15, 2024 09:21
@apskhem apskhem marked this pull request as ready for review October 15, 2024 11:23
@stevenj
Copy link
Contributor

stevenj commented Oct 15, 2024

@jmgilman This adds general auto-complete to forge so the PR is raised against master.
Looks OK to me.
The devx command autocomplete will be added on top of this in the devx branch with the next iteration of work.

@jmgilman
Copy link
Collaborator

Cool, looks good to me.

@jmgilman jmgilman merged commit 5131d0f into master Oct 15, 2024
20 checks passed
@jmgilman jmgilman deleted the feat/devx-autocomplete branch October 15, 2024 14:16
jmgilman added a commit that referenced this pull request Oct 15, 2024
@jmgilman
Revert "feat: adds bash autocompletions (#72)"
5ecee74 
This reverts commit 5131d0f.
@jmgilman jmgilman mentioned this pull request Oct 15, 2024
Merged
jmgilman added a commit that referenced this pull request Oct 15, 2024
@jmgilman
Revert "feat: adds bash autocompletions (#72)" (#74)
c8ac149 
This reverts commit 5131d0f.
@apskhem apskhem restored the feat/devx-autocomplete branch October 17, 2024 18:04
@apskhem apskhem mentioned this pull request Oct 17, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jmgilman jmgilman jmgilman approved these changes

@stevenj stevenj Awaiting requested review from stevenj

Assignees

@apskhem apskhem

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Add autocompletion when looking up commands from the markdown definition
3 participants
@apskhem @stevenj @jmgilman