87 actually broke Fish = 2.7. _flag_editor is local to each if branch, and remains unset further down in the file. Funnily enough, it will start working again in fish v3, that's one of the breaking changes (if blocks no longer introducing local scope bump, because that was stupid).
This fixes it by shimming argparse to do nothing and delete itself if it doesn't exist already, so that argparse can run outside the if block.
87 actually broke Fish = 2.7. _flag_editor is local to each if branch, and remains unset further down in the file. Funnily enough, it will start working again in fish v3, that's one of the breaking changes (if blocks no longer introducing local scope bump, because that was stupid).
This fixes it by shimming argparse to do nothing and delete itself if it doesn't exist already, so that argparse can run outside the if block.