revwalk: add a failint test for pushing "tags"

This shows that pusing a whole namespace can be problematic.
revwalk: ignore wrong objec type in glob pushes

Pushing a whole namespace can cause us to attempt to push non-committish
objects. Catch this situation and special-case it for ignoring this.
revwalk: remove usage of foreach