Mislav Marohnić
7a91ba5942
Print usage help when args given to "NoArgs" commands
2021-09-22 19:51:12 +02:00
Mislav Marohnić
a55f7af92c
Correct wrong args constraints
2021-09-22 19:36:25 +02:00
Mislav Marohnić
32d3a38465
Name of the codespace
2021-09-22 16:39:50 +02:00
Mislav Marohnić
cb7b535b91
Add tests for delete
2021-09-22 16:11:34 +02:00
Mislav Marohnić
d2113e3b59
Merge branch 'mislav/delete-codespaces' into raffo/delete-codespaces
2021-09-21 21:10:29 +02:00
Mislav Marohnić
678da44c28
Simplify delete further
2021-09-21 21:09:26 +02:00
Mislav Marohnić
ab86739b6b
Merge remote-tracking branch 'origin' into mislav/delete-codespaces
2021-09-21 20:35:58 +02:00
Mislav Marohnić
090e0c81a1
Merge pull request #171 from github/cli-migration
...
Split out "main" package from "ghcs"
2021-09-21 18:06:06 +02:00
Mislav Marohnić
683d847dd2
Merge remote-tracking branch 'origin' into cli-migration
2021-09-21 17:38:41 +02:00
Jose Garcia
8a53c4369e
Merge pull request #179 from github/jg/close-session
...
liveshare: close sessions
2021-09-21 10:14:22 -04:00
Jose Garcia
e8e914c220
PR Feedback
...
- Upgrade to go-liveshare v0.19.0
- Remove export helper method
- Use local implementation
2021-09-21 10:05:48 -04:00
Alan Donovan
0f88081bfa
Merge pull request #178 from github/check-authorized-keys
...
ghcs ssh: check user has authorised SSH keys
2021-09-21 10:03:45 -04:00
Alan Donovan
d3d1ce726d
do logs too
2021-09-21 10:02:55 -04:00
Alan Donovan
f33d430500
Merge pull request #177 from github/delete-surplus-args
...
delete: reject positional arguments
2021-09-21 09:26:22 -04:00
Jose Garcia
a83b3c0816
Update to go-livesare v0.18.0
...
- Only set err if closeErr is non-nil
2021-09-21 08:46:32 -04:00
Jose Garcia
7f682f9c39
Close Live Share sessions
...
- New helper method codespaces.CloseSession to be used using defer
- Upgrade to go-liveshare v0.17.0
2021-09-20 16:56:57 -04:00
Alan Donovan
dbb80d8b1e
check for authorised SSH keys
2021-09-20 16:01:43 -04:00
Alan Donovan
9e08b7477d
delete: reject position args
2021-09-20 13:40:45 -04:00
Mislav Marohnić
b894d3e134
Simplify delete implementation
2021-09-20 18:37:00 +02:00
Raffaele Di Fazio
c222c3d696
drop check on shut down
...
Signed-off-by: Raffaele Di Fazio <raffo@github.com>
2021-09-20 18:23:00 +02:00
Mislav Marohnić
836075215d
Merge remote-tracking branch 'origin' into cli-migration
2021-09-20 13:59:32 +02:00
Jose Garcia
85f79ed8e8
Merge pull request #159 from github/jg/ssh-cmd-flags
...
ghcs ssh: ssh flags and command support
2021-09-20 07:43:16 -04:00
Raffaele Di Fazio
c4f0eda96d
force was actually needed by a next commit
...
Signed-off-by: Raffaele Di Fazio <raffo@github.com>
2021-09-20 11:54:30 +02:00
Raffaele Di Fazio
eca3ecb43b
Merge branch 'main' into raffo/delete-codespaces
2021-09-20 11:53:35 +02:00
Raffaele Di Fazio
4721e7004b
add threshold to delete by repo
...
Signed-off-by: Raffaele Di Fazio <raffo@github.com>
2021-09-20 11:10:44 +02:00
Raffaele Di Fazio
11024f71fa
force is not used in delete by repo
...
Signed-off-by: Raffaele Di Fazio <raffo@github.com>
2021-09-20 10:27:29 +02:00
Jose Garcia
82c19729d3
Wrap -- with optional argument brackets
2021-09-17 15:17:38 -04:00
Jose Garcia
47c6a5fce8
Update usage
2021-09-17 15:13:09 -04:00
Jose Garcia
5890d6ad66
Switch if block logic, assert err string
2021-09-17 15:04:55 -04:00
Jose Garcia
da58313358
Remove redudant type def
2021-09-17 14:03:31 -04:00
Jose Garcia
9f84015bd0
Avoid append
2021-09-17 14:00:16 -04:00
Jose Garcia
65e1c6f789
More test cases
2021-09-17 13:56:38 -04:00
Jose Garcia
76037ee753
Update docs, simplify loop to append to command
2021-09-17 13:54:00 -04:00
Jose Garcia
54265afda0
PR Feedback
...
- use named returns
- handle command flags + test case
- simplify tests
2021-09-17 13:43:23 -04:00
Jose Garcia
60d066f0a6
PR Feedback
...
- return nil for slices
- handle `-L -l` case
- document `parseSSHArgs`
2021-09-17 11:51:37 -04:00
Alan Donovan
67a6f0a85d
Merge pull request #168 from github/restore-delete-r-confirm
...
Restore confirmation to delete -r, lost in botched merge
2021-09-17 11:48:02 -04:00
Alan Donovan
1271e20b9b
Merge pull request #169 from github/delete-no-list
...
delete -r: don't perform "list" operation
2021-09-17 11:28:53 -04:00
Alan Donovan
930f194507
Merge pull request #170 from github/list-show-branch-not-name
...
list: show branch (not name) in branch column
2021-09-17 11:28:26 -04:00
Mislav Marohnić
8c0c7a8e19
Make GITHUB_TOKEN configurable through Go member
...
Co-authored-by: Jose Garcia <josebalius@github.com>
2021-09-17 16:29:35 +02:00
Mislav Marohnić
c2f3537a32
Separate "main" package from "ghcs" package
...
To make "ghcs" importable, this separates out the `main()` function into
its own package that lives under "cmd/ghcs/main". Typically the main
package would be called "cmd/ghcs", but we wanted to leave the current
ghcs implementation where it is to avoid causing conflicts with current
work in progress.
Co-authored-by: Jose Garcia <josebalius@github.com>
2021-09-17 16:26:20 +02:00
Alan Donovan
ce4bbe5bd8
list: show branch (not name) in branch column
2021-09-17 10:13:35 -04:00
Raffaele Di Fazio
23c9026f58
Merge branch 'main' into raffo/delete-codespaces
2021-09-17 16:13:12 +02:00
Alan Donovan
d23eca8c5f
remove "list" operation from "delete -r" command
2021-09-17 09:51:11 -04:00
Alan Donovan
747d7e7173
Restore confirmation to delete -r, lost in botched merge
2021-09-17 09:49:46 -04:00
Alan Donovan
6ac8a0ad88
Merge pull request #165 from github/delete-parallel
...
Delete in parallel
2021-09-17 09:34:10 -04:00
Alan Donovan
610ab89c3d
Merge branch 'main' into delete-parallel
2021-09-17 09:32:56 -04:00
Alan Donovan
4de4572813
add comment
2021-09-17 09:31:05 -04:00
Alan Donovan
fb197c8e75
Merge branch 'main' into delete-parallel
2021-09-17 09:30:48 -04:00
Raffaele Di Fazio
c6b5fb5ba3
add the tests
...
Signed-off-by: Raffaele Di Fazio <raffo@github.com>
2021-09-17 14:55:50 +02:00
Raffaele Di Fazio
054fec0ba1
address code comments
...
Signed-off-by: Raffaele Di Fazio <raffo@github.com>
2021-09-17 14:45:08 +02:00