Uh oh!
There was an error while loading. Please reload this page.
convert circleci workflows to github actions - #42931
Conversation
|
Base commit: fb42a55 |
NickGerleman
commented
Feb 9, 2024
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:
|
There was a problem hiding this comment.
| runs-on: ubuntu-latest | |
| runs-on: 8-core-ubuntu |
There was a problem hiding this comment.
| runs-on: macos-12 | |
| runs-on: macos-12-large |
robandpdx
commented
Feb 26, 2024
@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! |
cipolleschi
commented
Feb 28, 2024
/rebase - this command should automatically rebase this PR on top of main |
cortinico
commented
Mar 1, 2024
/rebase |
robandpdx
commented
Mar 5, 2024
@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 |
@cortinico@cipolleschiHere is the latest workflow run from my fork. All green now. It would be great to get this merged soon. |
cortinico
commented
Mar 7, 2024
Amazing work @robandpdx ! How do we want to split this work to start merging it? Should we do the Android first? |
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
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 |
cortinico
commented
Mar 8, 2024
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. |
robandpdx
commented
Mar 11, 2024
@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. |
cortinico
commented
Mar 12, 2024
Can we do the following:
|
robandpdx
commented
Mar 13, 2024
@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.
robandpdx
commented
Mar 26, 2024
@cortinico I have opened this PR with only the android build and test. |
react-native-bot
commented
Sep 22, 2024
This PR is stale because it has been open 180 days with no activity. Remove stale label or comment or this will be closed in 7 days. |
react-native-bot
commented
Sep 22, 2024
This PR is stale because it has been open 180 days with no activity. Remove stale label or comment or this will be closed in 7 days. |
cortinico
commented
Sep 23, 2024
Close as we already merged this in main |
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_rntesterjobs fails with errors.build_npm_packagejob 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-windowsjob fails with unit test errors. See the log for details.Questions
build_androidandbuild_npm_packagejobs 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