Kynan Ware
35d81e3ff5
Fix: Gist edit tests for interactivity
...
This changes the gist edit tests to use the positive `istty` instead of the previous inverse `nontty`, which is consistent with the way other commands are written.
2025-01-16 10:46:07 -07:00
Mateus Marquezini
4eb0e8138f
Merge branch 'trunk' into fix/issue_10042
2025-01-13 11:47:52 -03:00
Mateus Marquezini
c0f6eb0598
#10042 : Attempt to add a new test scenario for handling error messages when TTY is unavailable
2025-01-13 11:39:48 -03:00
Andy Feller
41e99b9eab
Merge pull request #9934 from cli/andyfeller/cli-cli-8183-timeout-pt2
...
Ensure extension update notices only notify once within 24 hours, provide ability to disable all extension update notices
2025-01-13 09:12:32 -05:00
Mateus Marquezini
977e2326a2
#10042 : Applied code review suggestions to view_test.go
2025-01-13 11:06:13 -03:00
Andy Feller
3ada5af872
Enhance help docs on ext upgrade notices
...
- update `gh help environment` to include that upgrade notices only happen when gh or extensions are executed
- update `gh ext --help` to include reference to upgrade notices and points to `gh help environment` for info on disabling
2025-01-13 08:41:10 -05:00
Andy Feller
8a2be52be0
Merge pull request #10225 from zhuhaicity/fix_func_name
...
chore: fix some function names in comment
2025-01-13 08:08:18 -05:00
Mateus Marquezini
24e9fed7e0
removed unnecessary space
2025-01-12 14:57:24 -03:00
Mateus Marquezini
c5d6ae6cf6
Update pkg/cmd/gist/edit/edit.go
...
Update pkg/cmd/gist/edit/edit.go after code review
Co-authored-by: Kynan Ware <47394200+BagToad@users.noreply.github.com>
2025-01-12 14:53:20 -03:00
Mateus Marquezini
f65ac161ee
Merge branch 'trunk' into fix/issue_10042
2025-01-12 14:50:41 -03:00
zhuhaicity
13211af256
chore: fix some function names in comment
...
Signed-off-by: zhuhaicity <zhuhai@52it.net>
2025-01-12 15:07:04 +08:00
Andy Feller
0a1c9be656
Expand docs on cleaning extension update dir
...
This summarizes a conversation within cli/cli#9934 PR regarding how the new extension update directory metadata would affect gh if not cleaned up.
2025-01-10 14:57:10 -05:00
Andy Feller
97eb8a194e
Simplifying cleanExtensionUpdateDir logic
...
This function was doing some unnecessary heavy lifting detecting if the directory being deleted actually existed when `os.RemoveAll()` would handle directories that exist or not.
2025-01-10 14:29:55 -05:00
Andy Feller
20062233c8
Separate logic for checking updates
...
During discussion in cli/cli#9934 , we can to the conclusion that the logic around checking for core GitHub CLI updates would diverge from GitHub CLI
extension updates over time. To that end, this commit splits that logic into a separate function with a new environment variable.
2025-01-10 14:16:46 -05:00
Andy Feller
1c32c986ec
Capture greater detail on updaterEnabled
...
This commit expands the in-line docs around updaterEnabled package variable used to affect release checking.
Along with clarifying specific details discovered when talking with @williammartin, I'm also removing a useless local variable.
2025-01-10 13:55:16 -05:00
Andy Feller
a6ea8fe4ed
Merge pull request #10216 from nobe4/improve-padded-display
...
fix: padded display
2025-01-10 13:45:51 -05:00
Kynan Ware
c0d41102a5
Merge pull request #10215 from cli/dependabot/go_modules/github.com/mattn/go-colorable-0.1.14
...
Bump github.com/mattn/go-colorable from 0.1.13 to 0.1.14
2025-01-10 11:17:21 -07:00
nobe4
69b17272d6
fix: add back colon that I removed
2025-01-10 17:17:14 +01:00
nobe4
69ab0c9c85
fix: actually read how MaxFunc work and simplify the code
2025-01-10 16:31:13 +01:00
nobe4
1899b83509
fix: padded display
...
Apply code suggestion from @andyfeller from:
- https://github.com/cli/cli/pull/10194#discussion_r1909761552
- https://github.com/cli/cli/pull/10194#discussion_r1909764278
2025-01-10 16:28:50 +01:00
William Martin
889c4c75f3
Collapse dryrun checks in ext bin upgrade
2025-01-10 16:06:01 +01:00
dependabot[bot]
567624b550
Bump github.com/mattn/go-colorable from 0.1.13 to 0.1.14
...
Bumps [github.com/mattn/go-colorable](https://github.com/mattn/go-colorable ) from 0.1.13 to 0.1.14.
- [Commits](https://github.com/mattn/go-colorable/compare/v0.1.13...v0.1.14 )
---
updated-dependencies:
- dependency-name: github.com/mattn/go-colorable
dependency-type: direct:production
update-type: version-update:semver-patch
...
Signed-off-by: dependabot[bot] <support@github.com>
2025-01-10 14:39:26 +00:00
Tyler McGoffin
4c8d124d37
Merge pull request #10194 from nobe4/fancier-extention-upgrade
...
feat: make the extension upgrade fancier
2025-01-09 14:59:52 -08:00
Andy Feller
21a1f8e8a9
Merge pull request #10208 from cli/andyfeller/10202-correct-issue-create-template-help
...
Correct help documentation around template use in `gh issue create`
2025-01-09 13:22:26 -05:00
Mateus Marquezini
4c2ba5681d
#10042 removed unnecessary field
2025-01-09 13:14:27 -03:00
Andy Feller
954d4d5dbc
Correct help documentation around template use in gh issue create
...
Relates #10202
While we figure out how to handle consistent experience using templates for creating issues and PRs, let's correct the help usage for issue template flag use as this is the issue template name, not filename.
2025-01-09 11:08:18 -05:00
Mateus Marquezini
f4f8840c3f
#10042 : Add error messages for 'gh gist view/edit' prompts when no TTY is detected
2025-01-09 12:07:15 -03:00
Andy Feller
6bbd70353b
Comment exported functions
2025-01-09 09:09:42 -05:00
Mateus Marquezini
a3a283ca83
Merge branch 'trunk' into fix/issue_10042
2025-01-08 09:08:41 -03:00
Tyler McGoffin
713346c736
Merge pull request #10196 from cli/jtmcg/update-FR-docs
...
Update triage.md to reflect FR experiment outcome
2025-01-07 16:10:49 -08:00
Tyler McGoffin
47c391b246
PR review edits
2025-01-07 15:42:17 -08:00
Tyler McGoffin
58ca3c69bb
Update triage.md to reflect FR experiment outcome
2025-01-07 10:36:15 -08:00
nobe4
442be87acb
Merge branch 'trunk' into fancier-extention-upgrade
2025-01-07 12:04:35 +01:00
nobe4
3b69ea6fbc
feat: make the extension upgrade fancier
...
This left-pad the extension names during upgrade so they all align
nicely. It makes easier to see what is happening.
2025-01-07 12:01:02 +01:00
Andy Feller
40984d1eb6
Merge pull request #10186 from cli/wm/clear-up-fine-grained-pats-auth-login
...
Clear up --with-token fine grained PAT usage
2025-01-06 15:24:37 -05:00
Andy Feller
b3be16e98e
Merge pull request #10190 from cli/andyfeller/go-gh-2.11.2-security-bump
...
Bump cli/go-gh for indirect security vulnerability
2025-01-06 14:38:49 -05:00
Andy Feller
757966ca7c
Bump cli/go-gh for indirect security vulnerability
2025-01-06 14:27:03 -05:00
Tyler McGoffin
1f4e005562
Merge pull request #10158 from aryanbhosale/trunk
...
fix(repo fork): add non-TTY output when fork is newly created
2025-01-06 10:11:14 -08:00
Tyler McGoffin
2358fcee83
Merge pull request #10177 from cli/cmbrose/pr-create-upstream-fix
...
Handle missing upstream configs for `gh pr create`
2025-01-06 09:32:00 -08:00
William Martin
c3b41e87b8
Panic mustParseTrackingRef if format is incorrect
2025-01-06 10:33:28 -06:00
William Martin
8b5073d617
Move trackingRef into pr create package
2025-01-06 10:33:28 -06:00
William Martin
62ecb1c84d
Make tryDetermineTrackingRef tests more respective of reality
...
Though it doesn't really matter, in practice upstream is always going to come before origin.
2025-01-06 10:33:28 -06:00
William Martin
57ba5e5608
Rework tryDetermineTrackingRef tests
2025-01-06 10:33:28 -06:00
William Martin
b8c167970b
Avoid pointer return from determineTrackingBranch
2025-01-06 10:33:28 -06:00
William Martin
27bd4b2aec
Doc determineTrackingBranch
2025-01-06 10:33:28 -06:00
William Martin
05764b8114
Don't use pointer for determineTrackingBranch branchConfig
2025-01-06 10:33:28 -06:00
William Martin
dc077dc09b
Panic if tracking ref can't be reconstructed
2025-01-06 10:33:28 -06:00
William Martin
3ae4e5da20
Document and rework pr create tracking branch lookup
2025-01-06 10:33:28 -06:00
William Martin
e7d49fb78c
Update pkg/cmd/auth/login/login.go
...
Co-authored-by: Andy Feller <andyfeller@github.com>
2025-01-06 17:21:02 +01:00
Kynan Ware
106d5d1144
Merge pull request #10181 from jsoref/issue-10084
...
Upgrade generated workflows
2025-01-06 09:08:33 -07:00