Skip to content

Split Fantom workflow into separate build and test jobs#54729

Open
andrewdacenko wants to merge 1 commit intofacebook:mainfrom
andrewdacenko:fantom-workflow-split
Open

Split Fantom workflow into separate build and test jobs#54729
andrewdacenko wants to merge 1 commit intofacebook:mainfrom
andrewdacenko:fantom-workflow-split

Conversation

@andrewdacenko
Copy link
Contributor

  • 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 added a commit to andrewdacenko/react-native that referenced this pull request Dec 3, 2025
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

Fails
🚫

📋 Verify Changelog Format - See Changelog format

Generated by 🚫 dangerJS against 56b052e

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.

@andrewdacenko andrewdacenko force-pushed the fantom-workflow-split branch from 56b052e to bd912b8 Compare March 5, 2026 19:09
andrewdacenko added a commit to andrewdacenko/react-native that referenced this pull request Mar 5, 2026
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


Reviewed By: cortinico

Differential Revision: D88012198

Pulled By: andrewdacenko
@github-actions
Copy link

github-actions bot commented Mar 5, 2026

Caution

Invalid Changelog Format

Please verify your Changelog format. See Changelog format

@andrewdacenko andrewdacenko force-pushed the fantom-workflow-split branch from bd912b8 to 8136ec3 Compare March 6, 2026 00:17
andrewdacenko added a commit to andrewdacenko/react-native that referenced this pull request Mar 6, 2026
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


Reviewed By: cortinico

Differential Revision: D88012198

Pulled By: andrewdacenko
@andrewdacenko andrewdacenko force-pushed the fantom-workflow-split branch from 8136ec3 to 12aa5d6 Compare March 6, 2026 11:03
andrewdacenko added a commit to andrewdacenko/react-native that referenced this pull request Mar 6, 2026
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


Reviewed By: cortinico

Differential Revision: D88012198

Pulled By: andrewdacenko
Summary:
Changelog: [Internal]

- 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


Reviewed By: cortinico

Differential Revision: D88012198
@andrewdacenko andrewdacenko force-pushed the fantom-workflow-split branch from 12aa5d6 to f2abaf3 Compare March 6, 2026 15:21
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. 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

Development

Successfully merging this pull request may close these issues.

4 participants