(In reply to Rob Wu [:robwu] from comment #1)
If the goal is the removal of the second argument altogether, why did the eslint rule from bug 1758474 only check for
This is something that we have been working on progressively, where the original aim was just bug 1609271, and these follow-ups only being realised recently. Just having bug 1609271 in itself helps with specific issues that I believe will help work towards es modules move forwards.
The introduction and enforcement of this check would be easier if you would ban the use of the second parameter, and either fix up all uses or add them to
.eslintrc for specific files (marking as warning or turning them off instead of treating them as an error).
There are indeed multiple ways that we could roll these remaining parts out / restrict them etc. In this case there's less than about 20 instances where the two-parameter arguments are still used, and I haven't been seeing new cases added, so my concern has been low.
When I filed this, my hope was to have time to go through those last instances and hopefully be able to clean them up and complete this within a few weeks. Then the job would have been done and the second parameter couldn't be used at all. Unfortunately I've just had two weeks of very high priority work which has stopped me getting to this stage.