Re-enable eslint in `commits.js` file

What does this MR do?

Reenables eslint in the commits.js file and rewrites it as an ES6 class.

Are there points in the code the reviewer needs to double check?

There's a bit of "dead" code that was removed as it was not found elsewhere in the project.

Why was this MR needed?

We had disabled eslint checks in this file and we wanted to reenable it to increase coverage of eslint.

Screenshots (if relevant)

N/A

Does this MR meet the acceptance criteria?

• Review
  • Has been reviewed by Frontend
• Conform by the merge request performance guides
• Conform by the style guides
• Squashed related commits together (didn't squash for easier review, will squash upon merge)

What are the relevant issue numbers?

Closes #42734 (closed)