Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Configure workflows to use proper run-names and for DRY #923

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Configure workflows to use proper run-names and for DRY #923

wants to merge 3 commits into from

Conversation

jmatsu
Copy link
Contributor

@jmatsu jmatsu commented Aug 23, 2023 •
edited
Loading

Issue

  • close N/a

Overview (Required)

  • Specify run-name for each workflow unless defined
  • Aggregate a couple of setup actions for iOS build into the single action

Links

  • N/a
Before After
ss 2023-08-24 at 2 47 12 ss 2023-08-24 at 2 43 41

2 samples may not show same jobs but enough for the comparison.

@github-actions
Copy link

Test Results

183 tests   183 ✔️  7m 2s ⏱️
  11 suites      0 💤
  11 files        0 ❌

Results for commit 592660f.

@github-actions github-actions bot temporarily deployed to deploygate-distribution August 23, 2023 17:54 Inactive
@jmatsu jmatsu marked this pull request as ready for review August 23, 2023 18:48
@jmatsu jmatsu requested a review from a team as a code owner August 23, 2023 18:48
@jmatsu jmatsu requested a review from ry-itto August 23, 2023 18:48
@takahirom
Copy link
Member

Thanks, it looks good, but I'm sorry, I maybe don't understand the merit. 🙇 It seems that the information we need is already available. Additionally, I would like to find an action by title. 👀
image

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

@ry-itto ry-itto Awaiting requested review from ry-itto ry-itto is a code owner automatically assigned from DroidKaigi/conferenceapp2023reviewer

At least 0 approving reviews are required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@jmatsu @takahirom