Skip to content

Conversation

@Yu-zh
Copy link
Collaborator

@Yu-zh Yu-zh commented Nov 12, 2025

Two changes:

  1. moon test --target all includes native backends now, so there's no need to run native backend separately
  2. add CI test with use_js_builtin_string on
  3. add native test in release mode on CI #2906 introduced redundant jobs, and this PR removed them.

And a refactor:
factored out some common testing script to actions/test/action.yml and actions/test-native-release/action.yml and share them across stable/pre-release/nightly CI workflows.

@Yu-zh Yu-zh requested a review from bzy-debug November 12, 2025 05:49
@coveralls
Copy link
Collaborator

coveralls commented Nov 12, 2025

Pull Request Test Coverage Report for Build 1989

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 91.383%

Totals Coverage Status
Change from base Build 1985: 0.0%
Covered Lines: 10170
Relevant Lines: 11129

💛 - Coveralls

@bobzhang bobzhang enabled auto-merge (rebase) November 12, 2025 06:57
@bobzhang bobzhang disabled auto-merge November 12, 2025 06:58
@bobzhang
Copy link
Contributor

hold on a bit, note we need a bit more discussions about this

@Yu-zh Yu-zh marked this pull request as draft November 12, 2025 08:21
@Yu-zh Yu-zh changed the title chore: update ci script ci: cleanup test script Nov 25, 2025
@Yu-zh Yu-zh requested review from myfreess and removed request for bzy-debug November 25, 2025 07:57
@Yu-zh Yu-zh marked this pull request as ready for review November 25, 2025 08:02
@Yu-zh
Copy link
Collaborator Author

Yu-zh commented Nov 25, 2025

@bobzhang This should be good now. All the testing jobs should take ~2mins to run.

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.

4 participants