{"thread":{"id":"50614","subject":"","startedAt":"2019-02-28T10:44:50Z","lastAt":"2019-02-28T10:44:50Z","messageCount":1,"participants":["Rohit Ashiwal"],"isPatch":false,"patchVersion":null,"patchTotal":null},"messages":[{"id":"370362","messageId":"CAL7ArXp0xJx4DLZ7jM78SRFPGE0Ph9itrZpV0LtQYhbART1s5g@mail.gmail.com","threadId":"50614","inReplyTo":null,"subject":"","fromName":"Rohit Ashiwal","fromEmail":"rohit.ashiwal265@gmail.com","sentAt":"2019-02-28T10:43:46Z","receivedAt":"2019-02-28T10:44:50Z","isPatch":false,"sender":{"key":"rohit.ashiwal265@gmail.com","avatar":"https://avatars.githubusercontent.com/u/31043830?v=4"},"body":"Hey Duy\nSorry for late reply.\n\n>\n> On the Git mailing list, Duy Nguyen wrote (reply to this):\n>\n> I was a bit worried that the \"test ! something\" could be incorrectly\n> converted because for example, \"test ! -d foo\" is not always the same\n> as \"test_path_is_missing\". If \"foo\" is intended to be a file, then the\n> conversion is wrong.\n>\n> But I don't think you made any wrong conversion here. All these\n> negative \"test\" are preceded by \"git rm\" so the expectation is always\n> \"test ! -e\".\n>\n\nYes, I thought about it earlier and made\nchanges thinking this only. Also when I was going through the code\nagain, I replaced other conditionals `[ -f ]` with test_path_is_file\netc.\n\nCiao\nRohit\n"}]}