feat: integrate pre-commit hooks

[AhmedGrati]

Description

This pull request adds the pre-commit hooks, and fixes our files to comply with the checks integrated into the pre-commit-config.yaml.

[cdrage]

@AhmedGrati I think something may have messed up with the commit? Looks like it's making changes that aren't about the pre-commit hook

[AhmedGrati]

@cdrage Thank you for pointing out this. In fact, I added pre-commit hooks, and apply these pre-commit hooks to the files so that all checks can pass. Please LMK if you see that it's appropriate to push the pre-commit config without changing files.

[k8s-ci-robot]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: AhmedGrati

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

• OWNERS [AhmedGrati]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[cdrage]

@cdrage Thank you for pointing out this. In fact, I added pre-commit hooks, and apply these pre-commit hooks to the files so that all checks can pass. Please LMK if you see that it's appropriate to push the pre-commit config without changing files.

Yeah, I think we should merge the other PR in first, then pre-commit files. But just waiting for the person to update his PR to remove the leftover JS files.

[AhmedGrati]

@cdrage I update it, can you review it please?

[cdrage]

LGTM!