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

verdi computer/code duplicate now reuses prepend_text and append_text #3788

Merged

Commits on Feb 21, 2020

  1. verdi computer/code duplicate now reuses prepend_text and append_text

    This required adding a new ContextualDefaultOption class
    to get a default callable that also receives the context.
    Moreover, a but was fixed in the function that prompts
    for the prepend/append_text, as if the file was not touched/modified,
    instead of reusing the text provided in input, an empty text was used instead.
    giovannipizzi committed Feb 21, 2020
    Configuration menu
    Copy the full SHA
    a6b5905 View commit details
    Browse the repository at this point in the history

Commits on Feb 26, 2020

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

Commits on Feb 27, 2020

  1. Modify tests to check append/prepend text duplication

    Sets prepend/append text when setting up the computer/code for the non-interactive tests.
    The correct duplication was already tested by an assertion, but the prepend and append texts were set to the default value of an empty string.
    ConradJohnston committed Feb 27, 2020
    Configuration menu
    Copy the full SHA
    77f7e0f View commit details
    Browse the repository at this point in the history