remove extra cls variable being passed

[RayBB]

I just noticed this and it seems like it's wrong...
Why would we pass cls when it should be passed automatically.

Technical

Testing

Screenshot

Stakeholders

[Copilot]

Pull request overview

This PR fixes a bug where an extra cls parameter was incorrectly passed to a classmethod call. When calling a classmethod on a class, Python automatically passes the class as the first argument, so explicitly passing cls as the first parameter results in the wrong arguments being passed to the method.

Changes:

• Fixed incorrect classmethod invocation in submit_delete_request by removing the redundant cls parameter

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[jimchamp]

The entire method should be deleted:

• It is not referenced in the codebase
• We have no notion of a "delete" type librarian request
• This method does not set an mr_type, which is used to determine the typ of librarian request (work merge, author merge, etc.)

[mekarpeles]

Agreed, whole method can be deleted @RayBB

[RayBB]

@mekarpeles the method is now deleted

[RayBB]

Given the broad agreement, this is now merged.