Skip to content

Conversation

@robandpdx
Copy link
Contributor

Summary:

This pull request converts the CircleCI workflows to GitHub actions workflows. This change only inlcudes the android build and test jobs.

Changelog:

Test Plan:

Here is the latest workflow run in my fork.


https://fburl.com/workplace/f6mz6tmw

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Mar 25, 2024
@github-actions
Copy link

github-actions bot commented Mar 25, 2024

Fails
🚫

📋 Verify Changelog Format - See Changelog format

Generated by 🚫 dangerJS against a5d75cc

@facebook-github-bot facebook-github-bot added the Shared with Meta Applied via automation to indicate that an Issue or Pull Request has been shared with the team. label Mar 25, 2024
@analysis-bot
Copy link

analysis-bot commented Mar 25, 2024

Platform Engine Arch Size (bytes) Diff
android hermes arm64-v8a 18,446,480 -430,798
android hermes armeabi-v7a n/a --
android hermes x86 n/a --
android hermes x86_64 n/a --
android jsc arm64-v8a 21,819,718 -430,479
android jsc armeabi-v7a n/a --
android jsc x86 n/a --
android jsc x86_64 n/a --

Base commit: 2205880
Branch: main

Copy link
Contributor

@cortinico cortinico left a comment

Choose a reason for hiding this comment

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

Great stuff 👍 I've left a couple of comments but then we can import this one

@robandpdx
Copy link
Contributor Author

robandpdx commented Mar 26, 2024

@facebook-github-bot
Copy link
Contributor

@cortinico has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

- name: Cache android build artifacts
uses: actions/[email protected]
with:
key: android-build-cache-${{ github.run_number}}
Copy link
Contributor

Choose a reason for hiding this comment

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

Shouldn't we specify also a restore key here?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Unless you need a different restore key, it is not needed.


echo ${{ steps.hermes-version.outputs.version }}
build_andriod:
runs-on: ubuntu-latest
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
runs-on: ubuntu-latest
runs-on: 8-core-ubuntu

Can we make this and other compilation heavy jobs on 8-core-ubuntu? This should be similar to the same machine we had on CircleCI

Copy link
Contributor Author

Choose a reason for hiding this comment

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

You could, however these larger runners would need to be configured in your org with the label 8-core-ubuntu in order to run. There are no runners with this label available by default.

@facebook-github-bot
Copy link
Contributor

@cortinico has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

@facebook-github-bot facebook-github-bot added the Merged This PR has been merged. label Apr 2, 2024
@facebook-github-bot
Copy link
Contributor

@cortinico merged this pull request in 036ffbc.

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

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. Merged This PR has been merged. Shared with Meta Applied via automation to indicate that an Issue or Pull Request has been shared with the team.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants