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

Experiment/with actual args #185

Merged
merged 4 commits into from
Oct 20, 2023
Merged

Experiment/with actual args #185

merged 4 commits into from
Oct 20, 2023

Conversation

kjuulh
Copy link
Contributor

@kjuulh kjuulh commented Oct 20, 2023

My old pr was closed by mistake: #177

@kjuulh kjuulh requested a review from a team as a code owner October 20, 2023 08:50
@kjuulh kjuulh force-pushed the experiment/with-actual-args branch 2 times, most recently from 341aca1 to 51472e1 Compare October 20, 2023 08:54
cmd/test.go Outdated Show resolved Hide resolved
kjuulh and others added 3 commits October 20, 2023 15:00
Signed-off-by: Kasper J. Hermansen <[email protected]>

feat: with no interactive instead

Signed-off-by: Kasper J. Hermansen <[email protected]>

Sets the interactive command as well as SHUTTLE_INTERACTIVE

interactive is surfaced both in cli as well as env variable

SHUTTLE_INTERACTIVE=true/default/false

default=false

Signed-off-by: Kasper J. Hermansen <[email protected]>

feat: introduce early parse flags to make sure we get root commands set before building run and ls

Signed-off-by: Kasper J. Hermansen <[email protected]>

feat: with help comment

Signed-off-by: Kasper J. Hermansen <[email protected]>

feat: fully compatible

Signed-off-by: Kasper J. Hermansen <[email protected]>

refactor: run subcommand

Signed-off-by: Kasper J. Hermansen <[email protected]>

docs: document sub run

Signed-off-by: Kasper J. Hermansen <[email protected]>

docs: more docs

Signed-off-by: Kasper J. Hermansen <[email protected]>

feat: make sure to print error properly

Signed-off-by: Kasper J. Hermansen <[email protected]>

feat: return named runCmd

Signed-off-by: Kasper J. Hermansen <[email protected]>

wip: still working on cmd help

Signed-off-by: Kasper J. Hermansen <[email protected]>

feat: fixed interactive mode

Signed-off-by: Kasper J. Hermansen <[email protected]>

feat: fix go mod

Signed-off-by: Kasper J. Hermansen <[email protected]>
Signed-off-by: Kasper J. Hermansen <[email protected]>
Signed-off-by: Kasper J. Hermansen <[email protected]>
@kjuulh kjuulh force-pushed the experiment/with-actual-args branch from 445d3e6 to f93cf0d Compare October 20, 2023 13:01
Co-authored-by: Martin Anker Have <[email protected]>
@kjuulh kjuulh merged commit 5fdda08 into master Oct 20, 2023
2 checks passed
@kjuulh kjuulh deleted the experiment/with-actual-args branch October 20, 2023 13:05
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