Skip to content

Commit

Permalink
Remove CLI option true defaults to enable negation (aptos-labs#5582)
Browse files Browse the repository at this point in the history
@wrwg @davidiw @gregnazario @alinush 

Per aptos-labs#5581 , DocGen CLI options defaulting to true prohibit negation.
  • Loading branch information
alnoki authored and areshand committed Dec 17, 2022
1 parent 50c3e8f commit 21664a8
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions aptos-move/framework/src/docgen.rs
Original file line number Diff line number Diff line change
Expand Up @@ -15,8 +15,8 @@ pub struct DocgenOptions {
#[clap(long)]
pub include_impl: bool,

/// Whether to include specifications in the generated documentation. Defaults to true.
#[clap(long, global = true)]
/// Whether to include specifications in the generated documentation. Defaults to false.
#[clap(long)]
pub include_specs: bool,

/// Whether specifications should be put side-by-side with declarations or into a separate
Expand All @@ -29,8 +29,8 @@ pub struct DocgenOptions {
pub include_dep_diagram: bool,

/// Whether details should be put into collapsed sections. This is not supported by
/// all markdown, but the github dialect. Defaults to true.
#[clap(long, global = true)]
/// all markdown, but the github dialect. Defaults to false.
#[clap(long)]
pub collapsed_sections: bool,

/// Package-relative path to an optional markdown template which is a used to create a
Expand Down

0 comments on commit 21664a8

Please sign in to comment.