Commit 0d99660
Add automatic gh auth login prompt when sharing benchmark results (#689)
* Add automatic gh auth login prompt when sharing results
- Modified setup_github_authentication to offer interactive gh auth login
- Added auto_login parameter to control whether to prompt for authentication
- If gh is not authenticated, prompts user to run gh auth login
- Runs gh auth login interactively if user agrees
- Updated share_results to pass through auto_login parameter
- Falls back to saving results locally if authentication fails or is skipped
This makes sharing results much easier - users no longer need to manually set up authentication beforehand. The function will guide them through the process when needed.
* Use gh_cli_jll for automatic authentication
- Added gh_cli_jll as a dependency
- Implemented get_gh_command() to select between system gh and JLL gh
- Updated all gh CLI calls to use the wrapper function
- Auto-login now works even when gh is not installed on the system
- Falls back to JLL-provided gh binary automatically
This ensures the auto-authentication feature works on all systems, regardless of whether gh CLI is installed.
---------
Co-authored-by: ChrisRackauckas <accounts@chrisrackauckas.com>1 parent 6abda9e commit 0d99660
File tree
2 files changed
+66
-24
lines changed- lib/LinearSolveAutotune/src
2 files changed
+66
-24
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
265 | 265 | | |
266 | 266 | | |
267 | 267 | | |
268 | | - | |
| 268 | + | |
269 | 269 | | |
270 | 270 | | |
271 | 271 | | |
272 | 272 | | |
273 | 273 | | |
274 | 274 | | |
275 | 275 | | |
276 | | - | |
| 276 | + | |
| 277 | + | |
| 278 | + | |
| 279 | + | |
| 280 | + | |
| 281 | + | |
| 282 | + | |
| 283 | + | |
| 284 | + | |
| 285 | + | |
| 286 | + | |
277 | 287 | | |
278 | 288 | | |
279 | 289 | | |
| |||
289 | 299 | | |
290 | 300 | | |
291 | 301 | | |
292 | | - | |
293 | | - | |
294 | | - | |
295 | 302 | | |
296 | 303 | | |
297 | 304 | | |
298 | 305 | | |
299 | 306 | | |
300 | | - | |
| 307 | + | |
301 | 308 | | |
| 309 | + | |
| 310 | + | |
| 311 | + | |
302 | 312 | | |
303 | 313 | | |
304 | | - | |
| 314 | + | |
305 | 315 | | |
306 | 316 | | |
307 | 317 | | |
| |||
314 | 324 | | |
315 | 325 | | |
316 | 326 | | |
317 | | - | |
318 | | - | |
319 | | - | |
320 | | - | |
321 | | - | |
| 327 | + | |
| 328 | + | |
322 | 329 | | |
323 | | - | |
| 330 | + | |
324 | 331 | | |
325 | 332 | | |
326 | | - | |
327 | | - | |
328 | | - | |
329 | | - | |
330 | | - | |
331 | 333 | | |
332 | 334 | | |
333 | 335 | | |
| |||
336 | 338 | | |
337 | 339 | | |
338 | 340 | | |
339 | | - | |
| 341 | + | |
| 342 | + | |
340 | 343 | | |
341 | 344 | | |
342 | 345 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
17 | 17 | | |
18 | 18 | | |
19 | 19 | | |
20 | | - | |
| 20 | + | |
21 | 21 | | |
22 | 22 | | |
| 23 | + | |
23 | 24 | | |
24 | 25 | | |
25 | | - | |
| 26 | + | |
26 | 27 | | |
| 28 | + | |
| 29 | + | |
| 30 | + | |
27 | 31 | | |
28 | | - | |
29 | 32 | | |
30 | 33 | | |
31 | 34 | | |
| |||
36 | 39 | | |
37 | 40 | | |
38 | 41 | | |
39 | | - | |
| 42 | + | |
40 | 43 | | |
41 | 44 | | |
42 | 45 | | |
| |||
48 | 51 | | |
49 | 52 | | |
50 | 53 | | |
51 | | - | |
| 54 | + | |
| 55 | + | |
| 56 | + | |
| 57 | + | |
| 58 | + | |
| 59 | + | |
| 60 | + | |
| 61 | + | |
| 62 | + | |
| 63 | + | |
| 64 | + | |
| 65 | + | |
| 66 | + | |
| 67 | + | |
| 68 | + | |
| 69 | + | |
| 70 | + | |
| 71 | + | |
| 72 | + | |
| 73 | + | |
| 74 | + | |
| 75 | + | |
| 76 | + | |
| 77 | + | |
| 78 | + | |
| 79 | + | |
| 80 | + | |
| 81 | + | |
| 82 | + | |
| 83 | + | |
| 84 | + | |
| 85 | + | |
| 86 | + | |
| 87 | + | |
| 88 | + | |
| 89 | + | |
| 90 | + | |
52 | 91 | | |
53 | 92 | | |
54 | 93 | | |
| |||
0 commit comments