Taking out the name and parameters makes sense. Those are already in the code.

I don't agree with taking out the descriptive parts because stored procedures can and will be pulled from the database. Without that information it's useless to have a quick look at a stored procedure to see what's been going on.

Something we add to our template is a section that shows how the sproc is called including the EXEC. It's a handy way to select and run it as well as do a copy-paste as needed.

Version control is great but think about how people dig into the database and work outside of version control even when they should know better.