Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

PackageVariable now returns the default on "true" #4591

Merged
merged 3 commits into from
Sep 11, 2024

Commits on Sep 4, 2024

  1. PackageVariable now returns the default on "true"

    In all doc versions until 4.8.0, PackageVariable had wording like:
    "The option will support the values yes, true, on, enable or search,
    in which case the specified default will be used", but the code didn't
    actually do that, it just returned True. With this change it now returns
    the default value, with a slight tweak - if the default is one of the
    spelled out enabling strigs, it returns the boolean True instead.
    
    The indication that the default is produced if a truthy string is given
    is restored to the manpage (it was never dropped from the User Guide).
    
    Signed-off-by: Mats Wichmann <[email protected]>
    mwichmann committed Sep 4, 2024
    Configuration menu
    Copy the full SHA
    afd59b1 View commit details
    Browse the repository at this point in the history
  2. New PackageVariable test - enter path as rawstring

    Else it fails on Windows.
    
    Signed-off-by: Mats Wichmann <[email protected]>
    mwichmann committed Sep 4, 2024
    Configuration menu
    Copy the full SHA
    3b4f74a View commit details
    Browse the repository at this point in the history

Commits on Sep 11, 2024

  1. Configuration menu
    Copy the full SHA
    ee49656 View commit details
    Browse the repository at this point in the history