pre-commit issue/doubt

[codiearyan]

I was working on this issue - https://github.com/PalisadoesFoundation/talawa-admin/issues/6572#issuecomment-3829344941

Now, I have fixed this issue, but while commiting the fix, the pre-commit hooks run linting on the full file, not just the modified lines. It has flagged multiple existing hardcoded values across the file.

Question: should I go ahead and replace these hardcoded values with their respective variables, and create a pr?

 

[palisadoes]

Include it as part of your PR

 

[Piyush-Goenka]

This check has been enabled to fix the issue of hardcoded value across the codebase.
This lint will now be checking entire staged files.
If you have any query regarding this, either ping me here or tag me in the issue/PR.