Respect job ID when passing it to the trace command

[cdloh]

Description

Reworked the RunTrace command so it takes a job instead of an SHA. It then respects that. View that passes in an SHA runs through RunTraceSha which then passes it the result onto RunTrace.

Related Issue

Resolves #766

How Has This Been Tested?

Thought I'd wait on feedback from this change first. If it's approved I'll take a punt at 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 change)
• Documentation
• Chore (Related to CI or Packaging to platforms)

[codecov]

Codecov Report

Merging #767 (399b13f) into trunk (2b76102) will not change coverage.
The diff coverage is 33.33%.

@@           Coverage Diff           @@
##            trunk     #767   +/-   ##
=======================================
  Coverage   58.47%   58.47%           
=======================================
  Files          93       93           
  Lines        6681     6681           
=======================================
  Hits         3907     3907           
  Misses       2395     2395           
  Partials      379      379           

Impacted Files	Coverage Δ
commands/ci/view/view.go	46.47% <0.00%> (ø)
commands/ci/trace/trace.go	58.97% <100.00%> (ø)

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2b76102...399b13f. Read the comment docs.

[zemzale]

LGTM! Nice work @cdloh