Skip to content

Add PreviousAttemptURL, RunAttempt, RunStartedAt field to WorkflowRun - #2238

Merged
gmlewis merged 1 commit into
google:masterfrom
tohjustin:issue-2237/add-new-fields-to-workflowrun
Dec 24, 2021
Merged

Add PreviousAttemptURL, RunAttempt, RunStartedAt field to WorkflowRun#2238
gmlewis merged 1 commit into
google:masterfrom
tohjustin:issue-2237/add-new-fields-to-workflowrun

Conversation

@tohjustin

Copy link
Copy Markdown
Contributor

Add PreviousAttemptURL, RunAttempt, RunStartedAt field to the WorkflowRun struct.

Fixes: #2237.

Fixes: google#2237.
Signed-off-by: Justin Toh <tohjustin@hotmail.com>
@codecov

codecovBot commented Dec 23, 2021

Copy link
Copy Markdown

Codecov Report

Merging #2238 (fb26aae) into master (63a1994) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@ Coverage Diff @@## master #2238 +/- ##
=======================================
Coverage 97.80% 97.80% =======================================
Files 113 113 Lines 10205 10205 =======================================
Hits 9981 9981 Misses 156 156 Partials 68 68 
Impacted FilesCoverage Δ
github/actions_workflow_runs.go100.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 63a1994...fb26aae. Read the comment docs.

@tohjustin

Copy link
Copy Markdown
ContributorAuthor

I've just signed the CLA, can one of the repository manager trigger a rescan? 😅 Thanks!

@gmlewisgmlewis left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you, @tohjustin !
LGTM.
Merging.

@gmlewis
gmlewis merged commit 4d7a0b5 into google:masterDec 24, 2021
@tohjustin

Copy link
Copy Markdown
ContributorAuthor

Thanks for reviewing this @gmlewis, happy holidays! 🙏🏼

@tohjustin
tohjustin deleted the issue-2237/add-new-fields-to-workflowrun branch December 24, 2021 10:37
@gmlewis

Copy link
Copy Markdown
Collaborator

Thanks for reviewing this @gmlewis, happy holidays! 🙏🏼

My pleasure! Merry Christmas and Happy Holidays to you too, @tohjustin !

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add new fields to WorkflowRun

2 participants

@tohjustin@gmlewis