upgrading minor challenge issues across the board with a batch script

formatting challenges has been updated (#34747) so <br> tags should no longer be required. Can anyone think of edge cases where a <br> may still be required?

variables declared with var could also be changed to const - or let if there is (re)assignment in the body. @ezioda004 pointed out there are challenges in ES6 section specifically aimed at changing var to const/let so miss that section.

[RandellDawson]

Suggest we run this when the number of pr’s returns to a handlable level.

I second this recommendation. There are a couple of PRs we are going to wait on creating (mass find/replace) PRs which affect 100s of files, so we have decided to wait until we get below 500 open PRs to avoid other PRs stacking up behind them as we clear through eariler PRs to prevent mass chaining of merge conflicts.

I think it is a great idea and some of this logic could be added to the Probot to catch these kinds of things when new PRs are created. When it gets closer to time to create the actual PR, we should revisit this issue to make sure we have everything covered.

[ezioda004]

Just to clarify - I meant, the challenges before the ES6 challenge Explore Differences Between the var and let Keywords should not be modified with let/const.