254 Commits

Author SHA1 Message Date
meatballhat a05e4d0982 Remove App.Compiled
Closes #753
2022-11-08 19:13:31 -05:00
Naveen Gogineni 13cc7677ab Add SkipFlagParsing to app 2022-11-02 09:23:54 -04:00
Naveen Gogineni fb3a9cebb4 FIx: Allow ext flags to be opt-out by default rather than opt-in 2022-11-01 17:44:12 -04:00
dearchap ae8d932413 Feature:(issue_269) Allow external package flag definitions (#1540)
* Feature:(issue_269) Add compatibility with external package flag definitions

* Add tests

* Add defer to remove global flag

* Use ptr to receiver for extFlag

* Add const for flag prefix to ignore
2022-10-28 09:17:13 -04:00
GuangyuFan 63cb372cce Feature: (issue_1451) customized slice flag separator (#1546)
* feat: customized slice flag separator

* feat: modify go doc

* feat: update unit test
2022-10-27 17:32:00 +08:00
Naveen Gogineni bb8ddda480 Cleanup: Fix app.go formatting 2022-10-14 17:26:56 -04:00
dearchap 33d08c39be Update app.go
Co-authored-by: Dan Buch <dan@meatballhat.com>
2022-10-14 02:46:00 -04:00
Naveen Gogineni 00afca42c8 Add RunAsSubcommand to keep API unchanged 2022-10-11 16:35:54 -04:00
Naveen Gogineni 4a109bc0ce Fix tests 2022-10-11 16:28:12 -04:00
Naveen Gogineni a9d6b0d821 Changes from code review 2022-10-11 09:18:57 -04:00
Naveen Gogineni 59095aa762 make cmd.Run use varargs 2022-10-10 18:32:56 -04:00
Naveen Gogineni 2047c6630c Cleanup 2022-10-09 16:49:30 -05:00
Naveen Gogineni bc9ae33465 Merge latest and fix tests 2022-10-09 16:49:30 -05:00
Naveen Gogineni d5947d7814 Initial cut 2022-10-09 16:49:30 -05:00
Naveen Gogineni 8227be1fe9 Add tests for command.VisibleFlagCategory 2022-10-07 11:06:34 -05:00
Wendell Sun 619958c3d2 Add ActionableFlag interface instead of modifying Flag interface directly 2022-09-16 00:07:14 +09:00
Wendell Sun 14366f7030 feat: flag action 2022-09-16 00:07:12 +09:00
Naveen Gogineni 8339b59554 Fix: Help name consistency among app/commands and subcommands 2022-09-11 10:21:56 -04:00
Ilia Choly ca9df40abd Add App.InvalidFlagAccessHandler (#1446)
* Add App.UnknownFlagHandler

* Rename App.UnknownFlagHandler to App.InvalidFlagAccessHandler

* Traverse parent contexts
2022-08-30 18:51:16 -04:00
Naveen Gogineni 57a8423422 Fix:(issue_1094) Dont execute Before/After handlers during shell completion 2022-08-17 21:19:34 -04:00
dearchap e3ee4fb3ef Merge pull request #1444 from Dokiys/fix/add_after_before_help_check
Fix After not run
2022-08-13 08:27:21 -04:00
Dokiy 1a1b9cd563 Fix HideHelp 2022-07-29 15:26:06 +08:00
Dokiy a1c26d5491 Fix After not run 2022-07-29 11:05:32 +08:00
James Alavosus 77feee843d Implement slightly wonky setup for checking against ...
subcommand names of a default command (should it be set)
2022-06-21 19:30:05 -04:00
James Alavosus 32dec1ddaa feature: add DefaultCommand field to App
See issue #1307 for context.
2022-06-21 19:28:59 -04:00
meatballhat 9de0cd3c4a Merge pull request #1396 from urfave/suggestion-pluggability
Introduce override hooks for suggestions
2022-05-22 11:14:46 -04:00
meatballhat a78717f4f9 Merge remote-tracking branch 'origin/main' into michaeljs1990-add-flag-category-support 2022-05-19 07:40:56 -04:00
meatballhat 5bb9f453dc Also make the did-you-mean template pluggable 2022-05-17 09:24:17 -04:00
meatballhat 68bd4903fd Introduce override hooks for suggestions
Related to https://github.com/urfave/cli/pull/1390#discussion_r871398659
2022-05-17 09:02:17 -04:00
meatballhat 63b1a7deee A few follow-up conflict resolutions 2022-05-07 14:43:15 -04:00
meatballhat 3d67b75be7 Merge branch 'suggestions' of ssh://github.com/saschagrunert/cli into saschagrunert-suggestions 2022-05-07 14:40:04 -04:00
meatballhat 16d5d5a3df Some changes per feedback in #1368 2022-05-06 22:23:17 -04:00
meatballhat 156eaafb22 Rework flag categories a bit
with internal maps instead of slices and slightly less public API
surface area
2022-04-30 08:59:50 -04:00
meatballhat a583a102fd Merge remote-tracking branch 'origin/main' into michaeljs1990-add-flag-category-support 2022-04-28 20:29:09 -04:00
meatballhat 9ce946162f Unshadow context package
and consistently name `*cli.Context` vars and method receivers `cCtx`
2022-04-23 22:07:16 -04:00
meatballhat e4580f0c50 Extend flag categorization to top-level (global) flags 2022-04-22 15:44:59 -04:00
meatballhat 75e4ee69e9 Porting remainder of #796 2022-04-22 15:01:40 -04:00
meatballhat 44958693a1 Merge branch 'add-flag-category-support' of ssh://github.com/michaeljs1990/cli into michaeljs1990-add-flag-category-support 2022-04-22 13:56:03 -04:00
meatballhat 7c50be3d7c Switch branch references from master to main
Connected to #1349
2022-04-21 16:14:27 -04:00
meatballhat 4b238b8ff0 Merge branch 'master' into suggestions 2022-04-19 16:45:20 -04:00
Naveen Gogineni 07e1fdf17d Cleanup context.go 2021-04-05 07:57:41 -04:00
Wlad Gumenyuk 57d60cd570 Fix: typo
Corrected "eror" to "error" on line 331.
2020-12-27 18:30:11 +01:00
Alexander Frolov e40b83a394 use Name as default to HelpName 2020-11-03 13:22:01 +03:00
Robert Liebowitz 30bb6982f2 Merge pull request #1191 from stellirin/master
feature: Add a App.Reader that defaults to os.Stdin
2020-10-21 21:49:32 -04:00
Charles Kenney e5407cccf7 fix grammatical error 2020-10-06 22:24:56 -04:00
Adam Farden 342ce5d654 feature: Add a App.Reader that defaults to os.Stdin
Closes: #1190
2020-10-02 12:39:44 +02:00
Aloïs Micard 33744eb004 Harmonize BeforeError handling 2020-04-27 19:50:15 +02:00
Sebastian Malton fed64f3ad7 add tests 2020-04-01 09:17:59 -04:00
Sebastian Malton 22b6dbaad3 Set App.ErrWriter in App.Setup()
- Defaults to os.Stderr
- Remove the App.errWriter() function since it is no longer needed
2020-03-31 10:43:36 -04:00
Robert Liebowitz 959cf9de8a Update docs and tests around cli.Exit
Some of the docs were lacking or incomplete concerning how to properly
use cli.Exit, cli.HandleExitCoder, and the ExitErrHandler field on cli.App.
This change aims to clarify the usage of those pieces.

I also noticed that we have two identical functions now: cli.Exit and
cli.NewExitError. Since the latter seems to be more recent and less well
documented, I went ahead and marked it as deprecated so that we can keep
our public interface small.

Also added a missing test case for behavior that's been around a while
but was not documented or tested.

Related: #1089
2020-03-24 20:32:39 -04:00