self.clients was already a set. cannot subscript
What does this MR do and why?
Rollback 1 line where self.clients was already a set and we try to subscript a set of a set. The linter caught this and I thought I tested this, but I realize now there was never a case created where that particular line was called.
How to set up and validate locally
Upload a file and confirm there is no error.
MR acceptance checklist
This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.
-
Have you checked this doesn't interfere/conflict/duplicate someone elses work? -
Have you fully tested your changes? -
Have you resolved any lint issues? -
Have you assigned a reviewer? -
Have you applied correct labels?
Edited by Andrew