Skip to content
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

Merge CI fixes from main to v2. #1781

Merged
merged 5 commits into from
Jul 29, 2024
Merged

Merge CI fixes from main to v2. #1781

merged 5 commits into from
Jul 29, 2024

Conversation

MartinMinkov and others added 5 commits July 29, 2024 10:06
…ced caching

- Introduced a `Prepare` job to set up the repository, cache dependencies, count unit tests, and calculate chunks for parallel execution.
- Updated `Build-And-Test-Server` job to restore repository and dependencies from cache, run integration tests, and add results to the job summary.
- Added `Run-Unit-Tests` job to execute unit tests in parallel chunks, upload test results, and add them to the job summary.
- Enhanced `Build-And-Test-Web` job to cache Playwright browsers, run end-to-end tests, and upload E2E test artifacts.
- Refined `Release-on-NPM` and `Release-mina-signer-on-NPM` jobs to build and publish the project if the version has changed.
The unit tests have been removed from the CI test script. This change was likely made because the unit tests are now being run separately or have been moved to a different part of the CI pipeline. Removing them from this script helps to keep the CI process more focused and efficient.
…to run after all unit tests have completed successfully

refactor(build-action.yml): make Build-And-Test-Web job depend on Prepare job instead of directly on unit tests for better separation of concerns and maintainability
@shimkiv shimkiv merged commit d1e2492 into v2 Jul 29, 2024
22 checks passed
@shimkiv shimkiv deleted the fix/v2-ci branch July 29, 2024 08:30
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.

3 participants