Hi Rohit, On Sun, Feb 24, 2019 at 11:08 AM Rohit Ashiwal <rohit.ashiwal265@xxxxxxxxx> wrote: > If you disregard that contribution, here is my approach to solving > this microproject: search for `test -f` in `t/**` and replace all > occurrences with `test_path_is_file` and similarly for other mentioned > commands. Is it correct? In addition to what Dscho (alias Johannes Schindelin) wrote, please just send one patch that replaces `test -d`, `test -d` and similar code in only one t/tXXXX-*.sh file. No need to do more than that. Thanks, Christian.