Question : when should procs have public role

would it be good practise to give public permission, when the proc is doing only selects, and not to have that role for insert/delete/updates?

any other criteria to judge this?

thanks

Answer : when should procs have public role

private information (for example, if your SP returns Salaries, SIN, ...)