fix: Subchannel Posting Should Not Error Entire Process

[therynamo]

Description

#99 was supposed to fix the issues with error handling, but it turns out the errors were not bubbling as expected and we were still getting unhandled promise rejections. It was clear that we didn't need to bomb the process if the message sending failed, since we run a post to comment routine after posting. So this PR removes unnecessary error throwing and provides clearer messaging.

Motivation and Context

^

How Has This Been Tested?

• unit tests?

Screenshots (if appropriate):

Types of changes

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to not work as expected)
• Chore (updating repository quality of life, like deps, linting, etc.)

Checklist:

• My code follows the code style of this project.
• My change requires a change to the documentation.
• I have updated the documentation accordingly.