Pass in the providerID to DeleteEntityByName

[jhrozek]

we're calling DeleteEntityByName to get rid of closed PRs. At that point we have the providerID but we're not passing it to the DB Delete function.

[evankanderson]

Scenario:

You might have the same name in two different providers, so we should require the provider information when deleting by name. e.g. if you have a mindersec in a gitlab provider and also in a github provider, deleting ("mindersec") might do an unintended delete.