Task automation and test improvement

[unlikelyzero]

Closes: #4423

Describe your changes:

This ticket attempts to address some of the issues we've been having with missed or slipped comments or testing notes. As the team has grown, our change velocity has increased -- and so have the associated number of PRs/Issues.

In the first implementation, we will not be blocking PRs -- just informing of failure.

Note: there is currently a bug with prcop which prevents it from picking up changes to the main config file on branches vs master. That is being tracked here makaroni4/prcop#3

This will work after merge to master

All Submissions:

• Have you followed the guidelines in our Contributing document?
• Have you checked to ensure there aren't other open Pull Requests for the same update/change?
• Is this change backwards compatible? For example, developers won't need to change how they are calling the API or how they've extended core plugins such as Tables or Plots.

Author Checklist

• Changes address original issue?
• Unit tests included and/or updated with changes?
• Command line build passes?
• Has this been smoke tested?
• Testing instructions included in associated issue?

Reviewer Checklist

• Changes appear to address issue?
• Changes appear not to be breaking changes?
• Appropriate unit tests included?
• Code style and in-line documentation are appropriate?
• Commit messages meet standards?
• Has associated issue been labelled unverified? (only applicable if this PR closes the issue)
• Has associated issue been labelled bug? (only applicable if this PR is for a bug fix)
• List of Acceptance Tests Performed. These will be turned into e2e tests.

[codecov]

Codecov Report

Merging #4673 (50c3f23) into master (ba2a603) will decrease coverage by 0.07%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #4673      +/-   ##
==========================================
- Coverage   56.44%   56.37%   -0.08%     
==========================================
  Files         718      718              
  Lines       22813    22813              
  Branches     1659     1659              
==========================================
- Hits        12876    12860      -16     
- Misses       9530     9545      +15     
- Partials      407      408       +1     

Impacted Files	Coverage Δ
src/plugins/imagery/components/ImageryView.vue	36.76% <0.00%> (-3.74%)	⬇️
src/api/objects/ObjectAPI.js	89.30% <0.00%> (-2.14%)	⬇️
src/ui/components/ObjectView.vue	44.02% <0.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 ba2a603...50c3f23. Read the comment docs.

[akhenry]

This looks really awesome, and adds a lot of value. Two small suggestions inline. I also want to make sure @jvigliotta takes a look at this.

[jvigliotta]

@unlikelyzero Maybe not a check-box, but can we mention something to the effect of "If applicable, any outside contributors must have a signed CLA on file."?

[unlikelyzero]

@jvigliotta spun that out to automation here #4739

[shefalijoshi]

Once the CLA comment has been addressed, this is good to merge.

Reviewer Checklist

• Changes appear to address issue?
• Changes appear not to be breaking changes?
• Appropriate unit tests included?
• Code style and in-line documentation are appropriate?
• Commit messages meet standards?
• Has associated issue been labelled unverified? (only applicable if this PR closes the issue)
• Has associated issue been labelled bug? (only applicable if this PR is for a bug fix)
• List of Acceptance Tests Performed.