I prefer code to be as assertive as possible about structure and constants if possible. If something in the function itself makes the assumption that lists are passed in
Well the thing is there’s no value in doing this redundantly. Repo.delete_all already checks that the inputs make sense. In fact the whole function is pretty redundant because Repo.delete_all is smart enough to no-op an empty list.
