Skip to content

Conversation

@sovdeeth
Copy link
Member

@sovdeeth sovdeeth commented Dec 10, 2025

Problem

Now that we're supporting 18 months of versions on Java 21, the test times are getting overly long since it's all a single action (7+ minutes).

Solution

This pulls from experiments done in #8254 and adds a test task (customTest) that runs tests based on provided properties rather than hardcoded values. This allows the github action parallelize-tests to create a matrix of test environments split over N actions, which then runs a subset of the total environments. Given each test takes about 60s, plus the initial build that takes 30s, this cuts runtime for 8 versions from 8:30 to 3:30 if split 3 ways, or 2:30 if split 4. I've chosen split 3 ways to not clog up the check tab too much.

Also added 1.21.1 test env for full coverage

Testing Completed

Manual confirmation of the customTest task working, plus the check tab of this pr

Supporting Information


Completes: none
Related: none
AI assistance: none

@sovdeeth sovdeeth requested review from a team as code owners December 10, 2025 18:14
@sovdeeth sovdeeth added the enhancement Feature request, an issue about something that could be improved, or a PR improving something. label Dec 10, 2025
@sovdeeth sovdeeth requested review from Absolutionism and UnderscoreTud and removed request for a team December 10, 2025 18:14
@sovdeeth sovdeeth added the github related Issues or Pull requests that affect the GitHub organization/repositories label Dec 10, 2025
@sovdeeth sovdeeth moved this to In Review in 2.14 Releases Dec 10, 2025
@skriptlang-automation skriptlang-automation bot added the needs reviews A PR that needs additional reviews label Dec 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement Feature request, an issue about something that could be improved, or a PR improving something. github related Issues or Pull requests that affect the GitHub organization/repositories needs reviews A PR that needs additional reviews

Projects

Status: In Review

Development

Successfully merging this pull request may close these issues.

2 participants