Am 05.12.2016 um 22:33 schrieb Leo Famulari:
> +             (substitute* "testing/test_argcomplete.py"
> +               (("def test_remove_dir_prefix" line)
> +                (string-append "@pytest.mark.skip"
> +                               "(reason=\"Assumes that /usr exists.\")\n    "
> +                               line)))

These trailing spaces in the replacement string are a bit irritating.
Perhaps its better to search for "^\s+def …" and replace with "   
@pytest.skip…" (leading spaces here). I don't know.

Otherwise LGTM.

-- 
Regards
Hartmut Goebel

| Hartmut Goebel          | h.goe...@crazy-compilers.com               |
| www.crazy-compilers.com | compilers which you thought are impossible |


Reply via email to