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
Jose Garcia
0b68aaab7e
Return error on 202 responses
...
- Start implementing the retry/poll flow
2021-09-21 09:59:16 -04:00
Jose Garcia
7765628033
Merge pull request #21 from github/jg/err-context
...
Add error context to Session.Close
2021-09-21 09:56:27 -04:00
Jose Garcia
5f6b3a5eee
Add error context to Session.Close
2021-09-21 13:46:30 +00: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
Mislav Marohnić
2c43436c7a
Merge pull request #4347 from danburzo/trunk
...
Add 'git+https' to list of supported URL protocols
2021-09-21 14:40:19 +02:00
Jose Garcia
5e9382e8b4
Merge pull request #20 from github/jg/close-session-v2
...
Close RPC conn only
2021-09-21 07:54:04 -04:00
vilmibm
1f3b872859
test for unsupported platform
2021-09-20 17:17:30 -05:00
vilmibm
e85b0480e9
track installed tag name
2021-09-20 17:10:18 -05:00
vilmibm
0e2861a507
WIP refactoring
2021-09-20 17:05:19 -05:00
vilmibm
f5d269ebad
WIP refactoring
2021-09-20 17:02:34 -05:00
vilmibm
af7805af53
WIP refactoring
2021-09-20 16:46:54 -05:00
vilmibm
f4d97dcedd
WIP refactoring
2021-09-20 16:25:26 -05:00
Jose Garcia
23f6d449e0
Close RPC conn only
...
- Only close SSH if RPC fails. Closing RPC automatically closes the
underlying stream which in this case is the SSH connection.
- I thought about closing the SSH conn instead of RPC, but there is a
bit more cleanup that the RPC library needs to do.
2021-09-20 21:16:54 +00:00
vilmibm
ae38daf08a
nit
2021-09-20 16:05:35 -05:00
vilmibm
b00e8a5681
more accurately check for binary extension
2021-09-20 16:02:20 -05: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
Jose Garcia
82d7733f43
Merge pull request #19 from github/jg/session-closer-tidy-up
...
Allow clients to Close a Session, general tidy up
2021-09-20 16:36:16 -04:00
Jose Garcia
40886479ae
Close SSH even if RPC Close fails
2021-09-20 20:35:12 +00:00
Alan Donovan
dbb80d8b1e
check for authorised SSH keys
2021-09-20 16:01:43 -04:00
Sam
02ed5a9666
Merge pull request #4316 from SiarheiFedartsou/sf-pulls-draft
...
Add `--draft` and `--non-draft` filters to `gh pr list`
2021-09-20 12:07:52 -07:00
Sam Coe
d14715f1e3
Convert bool to string early for pr list draft flag
2021-09-20 11:29:37 -07: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
Dan Burzo
9f43967042
Fixes #4346 : allow git+https URL protocol
2021-09-20 19:35:06 +03: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
Jose Garcia
57d04dc5f0
Allow clients to Close a Session, general tidy up
...
- Allow clients to call Close on a Session to clean up resources
- Switch to the %w verb for error wrapping
- Fix typo on Port struct after verifying the server does not have a
typo
2021-09-20 13:16:38 +00: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
Siarhei Fedartsou
6a34f53c6c
Change pr list --draft UX
2021-09-17 22:05:47 +03: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