Babak K. Shandiz
d129b94fc5
refactor: remove returned resp from api.EndpointNeedsScopes
...
The returned response from `api.EndpointNeedsScopes` causes `bodyclose`
linter to raise a false positive error, assuming it's a new response that
its body needs to be closed.
Signed-off-by: Babak K. Shandiz <babakks@github.com>
2025-11-03 20:05:24 +00:00
Shion Ichikawa
a7f14576aa
✅ (repo delete) remove comprehensive assertions on check stderr output
2025-10-29 08:47:34 +09:00
Shion Ichikawa
7083f6dda4
💡 (gh repo delete) Add warning when --yes is ignored without a repository, Closes : #12033
2025-10-29 03:13:51 +09:00
Copilot
5eddf8d523
Merge pull request #11536 from cli/copilot/fix-11535
...
Fix `gh repo delete --yes` safety issue when no repository argument provided
2025-08-20 11:03:44 -06:00
Azeem Sajid
69fff52026
Inconsistent format of description of flags (starting with lowercase letter) ( #10507 )
2025-02-27 11:40:21 +01:00
Azeem Sajid
8b7fb231ec
Remove trailing whitespace
2025-02-13 11:06:27 +05:00
Kynan Ware
74f13a9b4f
Apply suggestions from code review
...
Co-authored-by: Andy Feller <andyfeller@github.com>
2024-11-21 13:55:35 -07:00
bagtoad
677ed2cdcf
Refactor command documentation to use heredoc
2024-11-21 13:23:03 -07:00
Tyler McGoffin
81591a09b8
Use go-gh/auth package for IsEnterprise, IsTenancy, and NormalizeHostname
2024-10-15 11:56:43 -07:00
Alexander Seiler
218f29f296
Correct some typos
2023-04-21 03:51:31 +02:00
Sam Coe
dbc2f05124
Update go-gh to v2 ( #7299 )
...
* Update go-gh
* Update code for go-gh v2
2023-04-16 15:34:23 +10:00
Sam Coe
f7930a430b
Dont follow redirects on repo delete ( #7106 )
2023-03-14 04:38:57 +11:00
Damien Sedgwick
55053df592
repo delete: change confirmation flag to --yes ( #6911 )
2023-01-25 17:33:53 +01:00
Nate Smith
c13eb9e792
Merge pull request #6320 from cli/label-prompter
...
add and use ConfirmDeletion in {label,repo} delete
2022-10-04 14:49:59 -05:00
Mislav Marohnić
acb61072cd
Respect GH_HOST in auth login, repo delete
...
Bonus fix: respect fallback host argument to `FromFullNameWithHost()`
2022-09-26 13:32:54 +02:00
vilmibm
d8a276e83b
add and use ConfirmDeletion in {label,repo} delete
2022-09-21 17:19:13 -07:00
vilmibm
f7d5c5f725
move prompter to own package
2022-07-26 16:41:13 -05:00
vilmibm
4afb567d06
WIP: survey wrapper
2022-07-26 14:39:38 -05:00
Håvard Anda Estensen
58cb773e09
Replace ioutil with io and os ( #5498 )
2022-04-26 13:07:44 +02:00
Mislav Marohnić
44775f87c8
Add nolint directives to allow-list current lint violations
2022-01-14 19:52:52 +01:00
meiji163
a1426ac19b
implement samcoe's suggestions
2021-11-10 17:10:37 -08:00
meiji163
26748dc2f3
allow base repo notty
2021-11-02 10:59:30 -07:00
meiji163
09064a5f55
fix tests
2021-11-02 10:51:42 -07:00
meiji163
001b1a36e7
add base resolution to delete
2021-10-31 14:46:57 -07:00
Alan Donovan
f4491c7a80
Add FlagErrorf; encapsulate FlagError.error
2021-10-21 11:40:20 -04:00
Mislav Marohnić
b9c26dfba3
Declare that the delete repo endpoint needs delete_repo scope
...
The API endpoint doesn't declare this in its `X-Accepted-Oauth-Scopes`
response header.
2021-10-19 19:16:50 +02:00
Meijke
f7b4588d09
scope error handling
...
Co-authored-by: Mislav Marohnić <mislav@github.com>
2021-10-15 08:35:33 -07:00
meiji163
caa282f0ba
commit review suggestions
2021-10-14 21:26:31 -07:00
meiji163
548b39c140
add one test
2021-10-12 10:50:54 -07:00
meiji163
5d433021bd
simplify err handling and change flags
2021-10-12 10:16:14 -07:00
meiji163
9ea36de269
add tests
2021-10-08 20:32:07 -07:00
meiji163
a8028be8f9
add delete HTTP request and confirmation prompting
2021-10-07 09:53:22 -07:00
meiji163
025f86b911
init commit
2021-10-06 12:49:06 -07:00
