Skip to content

Conversation

@j-piasecki
Copy link
Contributor

Summary

Sets up automated testing of Hermes V1. The workflow will set up a new RN app from the template, enable Hermes V1, build the app, and run a simple E2E test using it.

E2E is based on the setup used in the React Native repository:


If I understand the repository setup correctly, to pick up the results of the actions and show them on the website, the action needs to upload GitHub artifacts in a specific format, which I believe the new workflow does. Please correct me if I'm wrong.

@@ -0,0 +1,206 @@
name: test-hermes-v1
description: Tests a library on a nightly
Copy link
Contributor

Choose a reason for hiding this comment

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

This should be adapted, I think

Copy link
Contributor

Choose a reason for hiding this comment

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

Also, a reusable workflow is probably better.

Copy link
Contributor

Choose a reason for hiding this comment

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

We can probably collapse this in the hermesv1.yml workflow, to be fair.

Given that there are some steps specific to Android and some other specific to iOS, we could even have a setup_maestro action that is shared and two different workflows: one for android and one for iOS

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The new setup is:

  • setup maestro action
  • setup app with Hermes V1 action
  • build & test android app
  • build & test ios app

I also moved collecting result to check-nightly workflow, so it's able to pick up results from library tests, as well as hermes v1.

@j-piasecki j-piasecki force-pushed the @jpiasecki/test-hermes-v1 branch from 9764245 to d89a744 Compare November 7, 2025 14:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants