-
Notifications
You must be signed in to change notification settings - Fork 24k
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
convert circleci workflows to github actions #42931
base: main
Are you sure you want to change the base?
convert circleci workflows to github actions #42931
Conversation
|
Base commit: fb42a55 |
Very exciting 🙂. @cortinico and @cipolleschi will be the experts on the existing setup, but I also took a quick look, and noticed a couple quick things:
|
packages/react-native-codegen/lib | ||
enableCrossOsArchive: true | ||
test_android: | ||
runs-on: ubuntu-latest |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
runs-on: ubuntu-latest | |
runs-on: 8-core-ubuntu |
.github/workflows/test-all.yml
Outdated
mkdir -p "$DEST_DIR" | ||
mv "hermes.framework.dSYM" "$DEST_DIR" | ||
test_ios_rntester_ruby_3_2_0: | ||
runs-on: macos-12 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
runs-on: macos-12 | |
runs-on: macos-12-large |
7228178
to
4d11b9b
Compare
@cortinico and @cipolleschi, I have updated this PR to only include the jobs that are succeeding in my fork. Please merge this as we discussed. I'll follow up with other PRs as I get more jobs working. Thanks again for the help! |
/rebase - this command should automatically rebase this PR on top of main |
/rebase |
44027d5
to
0983cca
Compare
@cortinico @cipolleschi I have rebased my branch today. Here is the latest workflow run in my fork. If you approve this workflow run, you should see the same results. |
So CircleCI jobs are currently red as we bumped packages on main to |
76490bb
to
0cc63a2
Compare
0cc63a2
to
16b4cdb
Compare
@cortinico @cipolleschi Here is the latest workflow run from my fork. All green now. It would be great to get this merged soon. |
Amazing work @robandpdx ! How do we want to split this work to start merging it? Should we do the Android first? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just spotting a few nits.
rename step to be more concise Co-authored-by: Alex Hunt <hello@alexhunt.io>
rename step Co-authored-by: Alex Hunt <hello@alexhunt.io>
@cortinico The |
Sure so a couple of pointers:
As the error message contains
which is suspicious. For dry-run we produce only
and do this: - if (exec('./gradlew publishAllToMavenTempLocal').code) {
+ if (exec('./gradlew publishAllToMavenTempLocal --info --stacktrace').code) { to add more logging. This can provide a bit more insights on why the job is failing at the moment. |
@cortinico I think it's best if I hand off this work to you or someone else who has the expertise to make it work. |
Can we do the following:
|
@cortinico I have given you write permission to my fork. In order to deliver small PRs, I would need a tool that can do stacked diffs across forks. I'm not aware of any such tool that supports stacked diffs across forks. The best I can do is get everything in this PR green. |
This reverts commit f6c240e.
@cortinico I have opened this PR with only the android build and test. |
Summary:
This pull request converts the CircleCI workflows to GitHub actions workflows.
The github actions workflow is meant to mimic this circleci workflow.
Errors
test_ios_rntester
jobs fails with errors.build_npm_package
job fails with an error indicating that the artifacts are not found. We may need to use upload-artifact and download-artifact rather than cache for these files.test-windows
job fails with unit test errors. See the log for details.Questions
build_android
andbuild_npm_package
jobs take a parameterrelease_type
. When and how does this get passed in?Test Plan:
These workflows were tested in my fork. Here are the latest workflow runs:
test-all
test-js
https://fburl.com/workplace/f6mz6tmw