refactor(credentials): rename set/clear to login/logout/clear
Problem: the `set` and `clear` subcommands don't clearly convey their intent — `set` reads like a generic setter rather than an auth action, and `clear` overloads single-platform and all-platform semantics in one subcommand. Solution: rename `set` to `login`, split `clear` into `logout` (per-platform, defaults to active) and `clear` (all platforms). Update parser, dispatcher, tab completion, and vimdoc.
This commit is contained in:
parent
18a60da2d8
commit
c689b3e462
4 changed files with 58 additions and 29 deletions
|
|
@ -104,7 +104,7 @@ end, {
|
|||
end
|
||||
return filter_candidates(candidates)
|
||||
elseif args[2] == 'credentials' then
|
||||
return filter_candidates({ 'set', 'clear' })
|
||||
return filter_candidates({ 'login', 'logout', 'clear' })
|
||||
elseif args[2] == 'race' then
|
||||
local candidates = { 'stop' }
|
||||
vim.list_extend(candidates, platforms)
|
||||
|
|
@ -126,7 +126,7 @@ end, {
|
|||
cache.load()
|
||||
local contests = cache.get_cached_contest_ids(args[3])
|
||||
return filter_candidates(contests)
|
||||
elseif args[2] == 'credentials' and vim.tbl_contains({ 'set', 'clear' }, args[3]) then
|
||||
elseif args[2] == 'credentials' and vim.tbl_contains({ 'login', 'logout' }, args[3]) then
|
||||
return filter_candidates(platforms)
|
||||
elseif args[2] == 'cache' and args[3] == 'clear' then
|
||||
local candidates = vim.list_extend({}, platforms)
|
||||
|
|
|
|||
Loading…
Add table
Add a link
Reference in a new issue