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.

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

Optimize CI a bit #232

Closed
wants to merge 11 commits into from
Closed

Optimize CI a bit #232

wants to merge 11 commits into from

Conversation

Goooler
Copy link
Contributor

@Goooler Goooler commented Mar 4, 2022

Follow up #230.

@Goooler Goooler changed the title Bump actions Use gradle-build-action Mar 4, 2022
Comment on lines 17 to 31
arch:
description: 'CPU architecture of the system image - x86, x86_64 or arm64-v8a'
required: false
default: 'x86'
emulator-options:
description: 'command-line options used when launching the emulator'
required: false
default: '-no-window -gpu swiftshader_indirect -no-snapshot -noaudio -no-boot-anim'
emulator-build:
required: false
description: 'build number of a specific version of the emulator binary to use'
channel:
description: 'Channel to download the SDK components from - `stable`, `beta`, `dev`, `canary`'
required: false
default: 'stable'
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@Goooler Goooler changed the title Use gradle-build-action Optimize CI a bit Mar 4, 2022
Comment on lines 56 to 62
- name: Cache Node
uses: actions/cache@v2
with:
path: ~/.npm
key: ${{ runner.os }}-node-${{ hashFiles('**/package-lock.json') }}
restore-keys: |
${{ runner.os }}-node-
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@ychescale9
Copy link
Member

Thanks for the PR, it's a bit hard to review with all the formatting changes. Feel free to create a new one with just action version bumps and create new issue to discuss other changes 👍

@ychescale9 ychescale9 closed this Apr 6, 2022
@Goooler Goooler mentioned this pull request Apr 6, 2022
@Goooler Goooler deleted the actions branch April 6, 2022 15:03
Sign up for free to 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.

2 participants