Skip to content

Split Fantom workflow into separate build and test jobs #54729

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
Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Split Fantom workflow into separate build and test jobs #54729

wants to merge 1 commit into from
+108 −59

Conversation

@andrewdacenko
Copy link
Contributor

@andrewdacenko andrewdacenko commented Nov 28, 2025

  • Created new build-fantom-runner action to compile the Fantom runner binary
  • Modified run-fantom-tests action to download and use pre-built binary
  • Updated test-all.yml workflow to run build and test as separate jobs
  • Removed build dependencies and ccache configuration from test job

Summary:

Changelog:

Test Plan:

@meta-cla meta-cla 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 Nov 28, 2025
@facebook-github-bot facebook-github-bot added p: Facebook Partner: Facebook Partner Shared with Meta Applied via automation to indicate that an Issue or Pull Request has been shared with the team. labels Nov 28, 2025
@meta-codesync
Copy link

meta-codesync bot commented Nov 28, 2025

@andrewdacenko has imported this pull request. If you are a Meta employee, you can view this in D88012198.

@andrewdacenko @facebook-github-bot
Split Fantom workflow into separate build and test jobs (facebook#54729)
56b052e 
Summary:
- Created new build-fantom-runner action to compile the Fantom runner binary
- Modified run-fantom-tests action to download and use pre-built binary
- Updated test-all.yml workflow to run build and test as separate jobs
- Removed build dependencies and ccache configuration from test job






## Changelog:

<!-- Help reviewers and the release process by writing your own changelog entry.

Pick one each for the category and type tags:

[ANDROID|GENERAL|IOS|INTERNAL] [BREAKING|ADDED|CHANGED|DEPRECATED|REMOVED|FIXED|SECURITY] - Message

For more details, see:
https://reactnative.dev/contributing/changelogs-in-pull-requests


Differential Revision: D88012198

Pulled By: andrewdacenko
@meta-codesync
Copy link

meta-codesync bot commented Dec 3, 2025

@andrewdacenko has exported this pull request. If you are a Meta employee, you can view the originating Diff in D88012198.

@react-native-bot
Copy link
Collaborator

react-native-bot commented Dec 3, 2025

Fails
🚫

📋 Verify Changelog Format - See Changelog format

Generated by 🚫 dangerJS against 56b052e

cortinico
cortinico approved these changes Dec 3, 2025
View reviewed changes
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.

Review automatically exported from Phabricator review in Meta.

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

Reviewers

@cortinico cortinico cortinico approved these changes

Assignees

No one assigned

Labels

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

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@andrewdacenko @react-native-bot @cortinico @facebook-github-bot