Refactor/time strings

[Keyrxng]

Resolves #14

Tasks to be carried out:

• related tests

I'm assuming this will be merged quickly and once it and #9 both are more specific test cases can be built if needed. I don't wish to replicate the same changes across PRs in regards to fixing the pr and issue fetching etc

Refactors:

• direct config access no need for timers and misc props
• isTaskStale uses ms same as config items and more readable

with config:

      - plugin: http://localhost:4000
        runsOn: ["issue_comment.created"]
        with:
          reviewDelayTolerance: "3 Days"
          taskStaleTimeoutDuration: "15 Days"
          maxConcurrentTasks: 3
          startRequiresWallet: false

QA:

• More than 15 days old
• Less than 15 days old

[0x4007]

I am confused what exactly is reviewDelayTolerance

[Keyrxng]

I am confused what exactly is reviewDelayTolerance

• reviewDelayTolerance: is used when there is no reviews on their open PRs so it's compared against the created_at value of the PR. Used in issue.ts => getAvailableOpenedPullRequests().
• taskStaleTimeoutDuration: this is for when the warning message should appear

[0x4007]

Warning message for a task being old and they should confirm the spec is still accurate? 30 days default.

The other should be either 3.5 days default (accommodate weekends) or 1 day. Whatever the original implementation has we can use.

[Keyrxng]

@0x4007 Month(s) is not a supported unit with ms and the original default was 1 for reviewDelay

reviewDelayTolerance: stringDuration({ default: "1 day" }),

[Keyrxng]

Bumping this for review as it also now resolves the issue exposed with the LogReturn logger export used in the bubble-up error comment