William Martin
d4cd79e28c
Use int64 for GitHub database IDs and add lint check
...
Change all struct fields representing GitHub database IDs from int to
int64 to match the API spec and prevent potential overflow on 32-bit
architectures.
Add a custom go/analysis linter (idtype-checker) that flags struct
fields with ID-like names or JSON tags using int instead of int64,
integrated into make lint.
Closes #9247
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
2026-05-13 13:16:43 +02:00
Caleb Brose
31bff5b5a8
Better handling for codespace state polling
2025-01-17 14:09:33 -06:00
William Martin
1d38230675
Move config interfaces into gh package ( #9060 )
2024-05-10 10:39:36 +02:00
William Martin
5007c18f5f
Fix unused params across project
2024-05-07 17:50:43 +02:00
David Gardiner
500f359592
Don't poll for addition codespace permissions if user chooses to create without them ( #8267 )
2023-10-30 13:22:38 -05:00
David Gardiner
a8f8728056
Poll permission acceptance endpoint on codespace creation
2023-10-12 16:55:13 -07:00
David Gardiner
64f4660ec7
Migrate all Codespaces operations from Live Share to Dev Tunnels ( #8149 )
...
* Migrate all Codespaces operations from Live Share to Dev Tunnels
* Remove Live Share references
* Fix linting errors
* Update comments, remove deps, add uint16 bound checks
* Fix tests and move keep-alive logic to forwarder
* Address comments
* Updated mock port forwarder
* Fix CodeQL error
* Update comment
* Update func name
* Add missing connection close
* Fix linting error
* https -> http
* Update defer
* Fix tests
2023-10-12 22:16:36 +00:00
David Gardiner
e059f32aa5
Forward codespace ports over Dev Tunnels
2023-09-27 13:42:40 -07:00
azarashi
bb42fa07aa
codespace: Handle HTTP request retry interruption ( #7846 )
...
* codespace: Handle HTTP request retry interruption
* codespace: Make the error message more detailed
2023-08-18 07:16:06 -07:00
Sam Coe
4a57a812f5
Upgrade to Go 1.21 ( #7843 )
2023-08-16 10:37:58 -07:00
John Keech
93e1511bae
Codespaces: Use the host name from the logged in server for commands ( #7795 )
...
* Use the host name from the logged in server for codespace commands
* Fix existing tests
* Add tests for server url configuration
* Rename defaultApiUrl to defaultAPIURL and comment cleanup
* Switch to t.Setenv in codespaces api tests
* Switch to t.Setenv in other tests
* Support custom server in web flows for list and create
* Rename GetServerURL() to ServerURL()
2023-08-08 09:32:06 -07:00
David Gardiner
3c05c99c83
Address comments
2023-05-30 12:20:23 -07:00
David Gardiner
f6a5398bf9
Update fields to show
2023-05-26 22:59:28 +00:00
David Gardiner
99cd0af8b4
Check if running in a codespace
2023-05-26 15:41:34 -07:00
David Gardiner
54560c19dd
Add gh cs view command
2023-05-26 09:54:58 -07:00
Kousik Mitra
d0207a2ede
Add capability to filter codespaces by repo owner ( #7347 )
...
* Add capability to filter codespaces by repo owner
* Replace Flags with PersistentFlags
* Reword flag description
* Update test seed
* Add mutual exclusion
---------
Co-authored-by: Caleb Brose <5447118+cmbrose@users.noreply.github.com>
2023-04-27 10:42:08 -05:00
Alexander Seiler
218f29f296
Correct some typos
2023-04-21 03:51:31 +02:00
Sam Coe
35a24caed2
Standardize retry mechanism ( #7027 )
2023-03-02 23:06:30 +00:00
John Keech
791c7db632
Fix typo in has_uncommitted_changes for codespaces ( #6812 )
...
The REST API for codespaces returns the following schema:
```
"git_status": {
"ahead": 0,
"behind": 0,
"has_unpushed_changes": false,
"has_uncommitted_changes": false,
"ref": "main"
},
```
The CLI was looking for `has_uncommited_changes` in the response instead of `has_uncommitted_changes`, leading to it always reporting a `false` value which could be different than the API response.
2023-01-05 22:02:32 +00:00
Des Preston
4ade907ae8
Allow setting codespace display name during create ( #6739 )
...
Co-authored-by: Mislav Marohnić <mislav@github.com>
2022-12-19 17:10:34 +01:00
Ashwin Jeyaseelan
a756ffb1e8
Fix Codespace Creation Devcontainer bug ( #6389 )
2022-10-18 11:11:28 +03:00
JP Ungaretti
4a81e46c1a
Add rebuilding state
2022-10-07 22:05:16 +00:00
Caleb Brose
0c00ca911e
Clean up API interface
2022-09-16 22:25:11 +00:00
Caleb Brose
cb914c1873
Cleanup extra param, array declaration
2022-08-09 10:16:24 -05:00
Caleb Brose
5212fc8d5b
Error handling
2022-08-03 15:26:00 +00:00
Caleb Brose
f313953642
Initial pass
2022-07-16 20:38:23 +00:00
Mark Phelps
6014b31d03
Fix case where codespace created while in provisioning state causes panic
2022-07-01 13:08:44 -04:00
Mark Phelps
38ea62afba
Codespaces list by repo ( #5854 )
...
* WIP
* Allow to list codespaces by repo
* Tests
* Cleanup
* respond to PR feedback
* Address review comments
2022-06-28 13:34:04 +00:00
Mislav Marohnić
6e3689d58b
Apply UI polish around codespaces org-admin commands ( #5843 )
2022-06-22 20:35:56 +02:00
Ashwin Jeyaseelan
47f5abdb6f
8gitbrix/add delete org codespace ( #5827 )
...
Co-authored-by: Mislav Marohnić <mislav@github.com>
2022-06-22 18:28:09 +02:00
Greggory Rothmeier
2294f49d07
Add support for org admins to stop a members codespace ( #5812 )
2022-06-22 13:56:22 +02:00
Greggory Rothmeier
4b36dc031f
Add flag to list codespaces under an organization ( #5807 )
...
Co-authored-by: Mislav Marohnić <mislav@github.com>
2022-06-21 13:57:40 +00:00
Jake Shorty
832f35e000
PreFlight => BillableOwner for less confusion
2022-06-20 17:12:07 +00:00
Jake Shorty
ddbf8590e8
Fix error message and JSON keys
2022-06-19 18:28:33 +00:00
Jake Shorty
cad9a050be
Get basics working with codespaces pre-flight during creation
2022-06-16 15:42:29 +00:00
Jake Shorty
edf83af07a
Add Codespaces pre-flight request to API client
2022-06-15 21:17:04 +00:00
Mislav Marohnić
afb69d2d31
cs edit: 💅 implementation and tests
2022-06-01 17:18:17 +02:00
Sam Coe
9454ad3769
Change int to uint64 for large fields ( #5651 )
2022-05-17 06:37:55 +00:00
Greggory Rothmeier
06153770a9
Merge pull request #5504 from greggroth/gregg/retention-period
...
Add support for setting codespace retention-period
2022-05-10 18:21:55 +02:00
Håvard Anda Estensen
58cb773e09
Replace ioutil with io and os ( #5498 )
2022-04-26 13:07:44 +02:00
Jeff Hubbard
7678274464
Merge pull request #5076 from cli/multi-devcontainer
...
Multi devcontainer
2022-04-20 12:06:23 -07:00
Jeff Hubbard
2b57084bd0
PR feedback
2022-04-20 11:52:32 -07:00
Jack Meyer
ce0562c5f2
Add codespace open --web flag ( #5380 )
...
Co-authored-by: Luan Vieira <luanzeba@github.com>
Co-authored-by: Mislav Marohnić <mislav@github.com>
2022-04-11 19:09:27 +02:00
Jeff Hubbard
de9b5ad8f8
Merge branch 'trunk' into multi-devcontainer
2022-04-08 15:17:33 -07:00
Jake Shorty
6783509d3b
Merge pull request #5410 from cli/codespaces-location-optional
...
Make location an optional flag when creating codespaces
2022-04-07 10:58:39 -06:00
Jake Shorty
41719f7624
Location is an optional flag when creating codespaces
2022-04-05 20:19:19 +00:00
Amanda Lin
6490f658de
handle IdleTimeoutNotice
2022-04-05 10:27:26 -05:00
Jeff Hubbard
9554e522af
Change the way we parse list-devcontainers response
2022-03-28 14:24:57 -07:00
Jeff Hubbard
03b8b16585
Merge branch 'trunk' of ssh://github.com/cli/cli into multi-devcontainer
2022-03-28 13:59:01 -07:00
Gabe D
3a949203c4
Update Codespace permissions opt out param ( #5344 )
2022-03-25 12:12:14 +01:00