Fix placement of setTimeout

[savetheclocktower]

The command was hanging because setTimeout was being called over and over, rather than just as needed (because the tag wasn't available yet). The setTimeout started out in one wrong place and then I “fixed” it by moving the setTimeout to a different wrong place.

There is a separate issue here: the API calls to GitHub could fail for any number of reasons. One reason they failed for me was because I exceeded the unauthenticated rate limit! This command should at least handle the error cases, but I believe it can also use the very auth token we use for the package backend in order to enable a much higher rate limit. Both of those are worth doing, but I'll save them for a different PR.

[DeeDeeG]

Tested working, thank you very much!

To clarify the issue / expected for anyone just catching up about this problem:

Issue: ppm publish succeeds but never exits.

Expected: ppm publish should succeed and exit, returning the CLI to the user's control for further commands.

Fix: Allows ppm publish to exit as it should after it finishes.

Main test method: Ran ppm publish patch with/without this change to confirm the fix.

Also tried some invalid things to have ppm publish exit with errors, and that worked as it should too.