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

Update vibe command to use new Button API #19

Closed
wants to merge 7 commits into from

Conversation

rob-3
Copy link
Contributor

@rob-3 rob-3 commented Jul 24, 2021

This won't work until this is merged. I mainly opened this to see if people like the new API or have any suggestions.

rob-3 added 6 commits July 24, 2021 18:45
Over time, there will be a large number of arguments to the run function
that inject various dependencies. Not all commands will use all the
dependencies, so using the destructuring syntax gives an easy way for
command authors to specify only what they need. This is a one time
breaking change that is fixed by this commit.
@rob-3
Copy link
Contributor Author

rob-3 commented Aug 1, 2021

Closed in favor of #25.

@rob-3 rob-3 closed this Aug 1, 2021
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.

1 participant