Fix ESLint pipeline

@ankitv18 when we have already created a separate ticket to fix eslint then its better to fix the issue rather than skipping it

Working on it.

I have resolved all eslint errors but it is not reflecting in the pipelines since the tests are failing after i resolved the merge conflicts.
Please review.

Looks like there are still merge conflicts, can this be rebased?

Changes looks good to me and verified locally too, hence RTBC'ed

This isn't fixed, because ESLint still isn't running on CI. From the issue summary:

Remove SKIP_ESLINT from gitlab ci

Oh, and we need to target the merge request at the 2.0.x branch. 1.x is no longer under development.

I think due to the rebase the changes were lost from gitlab CI file, may be the rebase was incorrect.

Pipeline is now green, hence moving in review.

Looks good, hence moving into RTBC