Add posting to pull-request review comment capability

[gentlementlegen]

The SDK should be able to post to pull-request threads. The API is a bit different for these as it should directly use pull from the REST API.

What happens right now:

• if "pull_request" is present in the payload, try to post to the pull-request itself
• if we wanted to post to the thread, we would need to use octokit.rest.pulls.createReplyForReviewComment or createReviewComment
• same thing goes for updating the comment that needs a different API
• maybe we could consider having an extra argument in the options to specify a pull-request comment ID

[gentlementlegen]

/start

[ubiquity-os-beta]

Beneficiary

0x0fC1b909ba9265A846b82CF4CE352fc3e7EeB2ED

Tip

• Use /wallet 0x0000...0000 if you want to update your registered payment wallet address.
• Be sure to open a draft pull request as soon as possible to communicate updates on your progress.
• Be sure to provide timely updates to us when requested, or you will be automatically unassigned from the task.

[ubiquity-os-beta]

Important

• Be sure to link a pull-request before the first reminder to avoid disqualification.
• Reminders will be sent every 3 days and 12 hours if there is no activity.
• Assignees will be disqualified after 7 days of inactivity.

[ubiquity-os-beta]

Caution

All linked pull requests must be closed to generate rewards.

[ubiquity-os-beta]

 [ 117.06 WXDAI ] 

@gentlementlegen

Contributions Overview

View	Contribution	Count	Reward
Issue	Task	1	100
Issue	Specification	1	17.06

Conversation Incentives

Comment	Formatting	Relevance	Priority	Reward
The SDK should be able to post to pull-request threads. The API …	6.63 content: content: p: score: 0 elementCount: 1 ul: score: 0 elementCount: 1 li: score: 0.5 elementCount: 4 result: 2 regex: wordCount: 91 wordValue: 0.1 result: 4.63	1	2	17.06

 [ 16.184 WXDAI ] 

@whilefoo

Contributions Overview

View	Contribution	Count	Reward
Review	Comment	5	16.184

Conversation Incentives

Comment	Formatting	Relevance	Priority	Reward
Otherwise it works fine:https://github.com/ubiquibot-whilefoo-…	10.32 content: content: p: score: 0 elementCount: 3 a: score: 5 elementCount: 2 result: 10 regex: wordCount: 4 wordValue: 0.1 result: 0.32	0.2	2	5.14
If I use `logger.ok("something", { first: 1, second: 2 }) …	1.28 content: content: p: score: 0 elementCount: 1 result: 0 regex: wordCount: 20 wordValue: 0.1 result: 1.28	0.8	2	2.624
I'm concerned that this can lead to some bugs. If I understand c…	2.64 content: content: p: score: 0 elementCount: 1 result: 0 regex: wordCount: 47 wordValue: 0.1 result: 2.64	1	2	6.86
Yeah it was like that from beginning but I've only noticed now	0.88 content: content: p: score: 0 elementCount: 1 result: 0 regex: wordCount: 13 wordValue: 0.1 result: 0.88	0.5	2	1.08
Maybe Context should become a class in the future	0.65 content: content: p: score: 0 elementCount: 1 result: 0 regex: wordCount: 9 wordValue: 0.1 result: 0.65	0.3	2	0.48