No, the point of the commit is to remove the call to reverselist().
Before this commit:
https://github.com/Baystation12/Baystation12/commit/0efb0311e33f4d51eefd0ab2b5aa35d5891b7604
reverselist did nothing.
That commit included some changes made to the tg codebase that fixed reverselist, however it did not include the change to MULEbot to remove the call to reverselist(which now has correct but different behaviour)
You can see the tg commit here:https://github.com/tgstation/-tg-station/commit/54d02d75a3da4ae3a73e00eea34d4bbfa3d6e73f .
As you can see it includes both the fix to reverse list and a clean up of the uses of reverselist(), which the commit to baystation did not.
Removing reverselist() is the important fix.

Further if you take a look at the AStar() code you can see that it provides the list of moves in the correct order, so it clearly makes no sense to reverse the list, It only worked before because reverselist() did nothing.