-
Notifications
You must be signed in to change notification settings - Fork 13
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
Add XSelectScheme command #23
Conversation
@@ -64,7 +65,7 @@ function! s:scheme() | |||
return '-scheme '. s:scheme_name() | |||
endfunction | |||
|
|||
function!s:scheme_name() | |||
function! s:scheme_name() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lol oops
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
😭
yep, that seems best.
Maybe we should add a small note that if you mistype the scheme, |
@@ -72,6 +73,11 @@ function!s:scheme_name() | |||
return s:chosen_scheme | |||
endfunction | |||
|
|||
function! s:set_scheme(scheme) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we move this up with build
and test
? I feel like it makes sense to group actions together, but maybe I'm insane?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nope that sounds good
Fixes #2 I'll open another issue to cover autocompletion. |
This command allows users to switch the scheme being built. When doing so we also reset the current SDK in case the scheme targets a different OS.
Updated for feedback! |
This command allows users to switch the scheme being built. When doing
so we also reset the current SDK in case the scheme targets a different
OS.
I pushed the xcpretty docs down since it made sense to group commands together. If you'd prefer this not change let me know and I can swap them.
I also didn't write much for the help since there's not a lot to this command, let me know if there's anything else I should include.