Add ListWorkflowJobsAttempt method to ActionsService

[fchimpan]

Fixes: #3059.

Add support ListWorkflowJobsAttempt API.
https://docs.github.com/rest/actions/workflow-jobs#list-jobs-for-a-workflow-run-attempt

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (20fc901) 97.71% compared to head (345d70e) 97.71%.

Additional details and impacted files

@@           Coverage Diff           @@
##           master    #3060   +/-   ##
=======================================
  Coverage   97.71%   97.71%           
=======================================
  Files         152      152           
  Lines       13326    13342   +16     
=======================================
+ Hits        13021    13037   +16     
  Misses        215      215           
  Partials       90       90           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[gmlewis]

Thank you, @fchimpan !
This looks like a great start apart from the code coverage.

I've added a suggestion that should hopefully clear up the coverage tests.

[gmlewis]

Thank you, @fchimpan !
LGTM

Awaiting second LGTM+Approval from any other contributor to this repo before merging.

[fchimpan]

@gmlewis

Thanks for your advice. I was already working on improving the test coverage before I noticed your comments, so there may have been some overlap. I would appreciate it if you could review the changes. I look forward to your feedback.

Best regards,

[gmlewis]

@fchimpan - while you are here, would you mind taking a look at other open PRs in this repo with the "Needs Review" label and reviewing them? I know that one person was waiting for #3049 in particular, but all of them need review at some point... feel free to pick and choose (or leave them alone - your choice).

[fchimpan]

@gmlewis
Thank you for the comment. I understand and am happy to help.
I have an interest in this repository, so I will definitely cooperate.

Since I don't have the authority to approve pull requests, should I just leave comments on the PR if I have any suggestions?

[gmlewis]

Thank you, @fchimpan !
Yes, feel free to leave comments, and when you are satisfied with the changes, you can say "LGTM" (Looks Good To Me) and I will take that as approval.

[fchimpan]

Thank you @gmlewis
Understood, I will check it out.

By the way, sometimes I come across APIs that are not yet implemented. In such cases, is it alright if I propose a PR like I did this time?

Thank you in advance.

[gmlewis]

By the way, sometimes I come across APIs that are not yet implemented. In such cases, is it alright if I propose a PR like I did this time?

Absolutely! We welcome all contributions.
The GitHub v3 API is amazingly huge and constantly updating, so we definitely welcome the help.
Thank you!

[valbeat]

LGTM!

[gmlewis]

Thank you, @valbeat !
Merging.