Fix the errors/warnings reported by PHP_CodeSniffer

Above error/warnings are fixed.

Charchil Khandelwal → made their first commit to this issue’s fork.

i will review this.

Patch #2 applied clearly, all the errors and warnings are fixed now.

Created MR !5 for this issue, please review.

Thank You.

Needs rebasing.

I updated the code with f68ff06627492e87fe6daa0988d6779b8c136a9e but it seems to be on the wrong branch.
Not sure of the best way to "undo" it and move to correct branch.

bindu r → made their first commit to this issue’s fork.

@tyler36 The branch is correct. The problem is that the project branch has been changed since the MR was created. I will see if I can fix it.

It seems I fixed the conflict. Now the MR can be used again.

As a side note, the commits that caused conflicts were added in 📌 Fix the issues reported by phpcs Fixed .

I re-run PHP_CodeSniffer for the project branch, and updated the report shown in the issue summary. There are just two warnings/errors that need to be fixed.

Needs reroll

(I am trying to rebase, but it seems to take too much.)

I've added gitlab CI to the project and I closed 📌 Get gitlab CI passing Closed: duplicate as a duplicate of this.

It would be good to get a green run before we merge this, including for phpcs and phpstan

Thanks