Skip to content

Select Windows test workers automatically - #22946

Merged
arnaud-lb merged 1 commit into
php:PHP-8.2from
NickSdot:perf/windows-auto-test-workers
Aug 10, 2026
Merged

Select Windows test workers automatically#22946
arnaud-lb merged 1 commit into
php:PHP-8.2from
NickSdot:perf/windows-auto-test-workers

Conversation

@NickSdot

@NickSdot NickSdot commented Jul 30, 2026

Copy link
Copy Markdown
Contributor

Extracted from #22917.

Removes the hard-coded -j2 from Windows CI so run-tests.php selects the worker count automatically. Depends on #22939 for automatic selection and #22957 for reliable concurrent subprocess execution.

@NickSdot

Copy link
Copy Markdown
Contributor Author

@arnaud-lb just wanna make your you didn't miss my reference to here in #22939 (comment)

Comment thread .github/workflows/test-suite.yml Outdated
@NickSdot
NickSdot force-pushed the perf/windows-auto-test-workers branch from 50d482b to 1ff0e94 Compare August 10, 2026 12:17
@NickSdot
NickSdot marked this pull request as draft August 10, 2026 12:20
@NickSdot
NickSdot changed the base branch from master to PHP-8.2 August 10, 2026 12:21
@NickSdot
NickSdot marked this pull request as ready for review August 10, 2026 12:23
@arnaud-lb
arnaud-lb merged commit 0d26c1c into php:PHP-8.2 Aug 10, 2026
18 checks passed
arnaud-lb added a commit that referenced this pull request Aug 10, 2026
* PHP-8.5:
  CI: Select Windows test workers automatically (#22946)
@arnaud-lb

Copy link
Copy Markdown
Member

Thank you!

pull Bot pushed a commit to wudi/php-src that referenced this pull request Aug 10, 2026
* PHP-8.2:
  CI: Select Windows test workers automatically (php#22946)
pull Bot pushed a commit to wudi/php-src that referenced this pull request Aug 10, 2026
* PHP-8.3:
  CI: Select Windows test workers automatically (php#22946)
pull Bot pushed a commit to wudi/php-src that referenced this pull request Aug 10, 2026
* PHP-8.4:
  CI: Select Windows test workers automatically (php#22946)
@NickSdot
NickSdot deleted the perf/windows-auto-test-workers branch August 10, 2026 17:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants