feat(wiby result): change result command output and exit codes

[rodion-arr]

Following #57 (comment) this PR introduces updates to wiby result command:

• Logs
  • debug package added as dependency
  • simple wrapper module for debug added
  • all console logs replaced with wrapper logger
  • logs for wiby test command enabled programmatically in order to save old outputs
  • each command got own log namespace (wiby:test, wiby:result, etc)
  • all logs can be enabled with DEBUG=wiby:* env
• wiby result
  • default export does not print intermediate logs now by default. Old logs can be enabled with DEBUG env
  • default export returns result object instead
  • based on returned object - relevant markdown is building (example)
  • failed checks will exit with code 1
  • pending checks will exit with code 64
  • result tests updated to check relevant output and exit code
  • default export returns object of next format:

{
   "status":"failure",
   "results":[
      {
         "dependent":"wiby-test/partial",
         "status":"pending",
         "runs":[
            ["Travis CI - Branch", null]
         ]
      },
      {
         "dependent":"wiby-test/fail",
         "status":"failure",
         "runs":[
            [ "Travis CI - Branch", "failure" ]
         ]
      },
      {
         "dependent":"wiby-test/pass",
         "status":"pending",
         "runs":[
             [ "Travis CI - Branch", null ]
         ]
      }
   ]
}

Related to #57

[ghinks]

If I am reading this correctly once we have completed iterating over all the runs and the isAllSuccess is still true.

Then following the methodology below should we return pipelineStatses.SUCCEED and change the final return to pipelineStatses.FAILED ?

[rodion-arr]

Agreed this will simplify code a bit, submitted changes

[ghinks]

I saw the changes you made in the getOverallStatusForAllRuns, many thanks.

Still reviewing. I took the main branch and merged back into your changes and created scoped package @gvhinks/[email protected]. I then created a PR on a dependency test package. The PR's associated action completed once I typed wiby result.

I took some screen shots of the actions with DEBUG=wiby:*

from an integration point of view its looking good.

[dominykas]

Looks good, aside from what I assume was a typo?

[dominykas]

🎉 This PR is included in version 0.2.0 🎉

The release is available on:

• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot 📦🚀