(assuming you're done with the update, we usually wait for the threads to be resolved)
the title of the third patch should be updated
Note: I'm fine with the logic of the MR as it is (ie accept it when points listed above are fixed). However, I think it could be further improved (up to you to move forward or not, in this MR or another one):
* the logic in changing the directory is currently overkill. we just need two cases: change current directory (+set env var) (the `cd /d` case or `cd` case in the same current directory) , or just change env var (if dir exists) (so no need in that case to change twice the current directory) * the logic in case of `cd foo*` is wrong (current code looks up for first directory in foo*, while native picks first element in foo*, and fails if it's not a directory)