Merge pull request #4310 from epage/skip

fix(derive): Support 'group' attribute with Parser

Author: epage

clap_derive/src/lib.rs

@@ -42,7 +42,7 @@ pub fn value_enum(input: TokenStream) -> TokenStream {
 /// receiving an instance of `clap::ArgMatches` from conducting parsing, and then
 /// implementing a conversion code to instantiate an instance of the user
 /// context struct.
-#[proc_macro_derive(Parser, attributes(clap, structopt, command, arg))]
+#[proc_macro_derive(Parser, attributes(clap, structopt, command, arg, group))]
 #[proc_macro_error]
 pub fn parser(input: TokenStream) -> TokenStream {
     let input: DeriveInput = parse_macro_input!(input);

tests/derive/groups.rs

@@ -62,6 +62,7 @@ For more information try '--help'
 #[test]
 fn skip_group_avoids_duplicate_ids() {
     #[derive(Parser, Debug)]
+    #[group(skip)]
     struct Opt {
         #[command(flatten)]
         first: Compose<Empty, Empty>,
@@ -88,7 +89,7 @@ fn skip_group_avoids_duplicate_ids() {
     use clap::Args;
     assert_eq!(Empty::group_id(), None);
     assert_eq!(Compose::<Empty, Empty>::group_id(), None);
-    assert_eq!(Opt::group_id(), Some(clap::Id::from("Opt")));
+    assert_eq!(Opt::group_id(), None);
 }
 
 #[test]