Skip to content

Add enterprise list runner applications download - #2496

Merged
gmlewis merged 1 commit into
google:masterfrom
gabriel-samfira:add-enterprise-list-runner-applications
Oct 16, 2022
Merged

Add enterprise list runner applications download#2496
gmlewis merged 1 commit into
google:masterfrom
gabriel-samfira:add-enterprise-list-runner-applications

Conversation

@gabriel-samfira

@gabriel-samfiragabriel-samfira commented Oct 14, 2022

Copy link
Copy Markdown
Contributor

This change adds ListRunnerApplicationDownloads() to the Enterprise service.

Fixes: #2495

Signed-off-by: Gabriel Adrian Samfira gsamfira@cloudbasesolutions.com

This change adds ListRunnerApplicationDownloads() to the Enterprise
service.
Signed-off-by: Gabriel Adrian Samfira <gsamfira@cloudbasesolutions.com>
@codecov

codecovBot commented Oct 14, 2022

Copy link
Copy Markdown

Codecov Report

Merging #2496 (c555ecc) into master (3344412) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@ Coverage Diff @@## master #2496 +/- ##
=======================================
Coverage 97.99% 97.99% =======================================
Files 123 123 Lines 10762 10774 +12 =======================================
+ Hits 10546 10558 +12 
Misses 148 148 Partials 68 68 
Impacted FilesCoverage Δ
github/enterprise_actions_runners.go100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@gmlewisgmlewis added the NeedsReview PR is awaiting a review before merging. label Oct 15, 2022

@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, @gabriel-samfira !
LGTM.

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

@valbeatvalbeat left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM!

@gmlewis

Copy link
Copy Markdown
Collaborator

Thank you, @valbeat !
Merging.

@gmlewisgmlewis removed the NeedsReview PR is awaiting a review before merging. label Oct 16, 2022
@gmlewis
gmlewis merged commit 38d7b16 into google:masterOct 16, 2022
jlaportebot added a commit to jlaportebot/go-github that referenced this pull request Jun 28, 2026
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.

Missing ListRunnerApplicationDownloads() for Enterprise

3 participants

@gabriel-samfira@gmlewis@valbeat