Skip to content

Make integration tests more stable in CI#19995

Merged
RobinMalfait merged 2 commits into
mainfrom
fix/improve-ci-integrations
Apr 30, 2026
Merged

Make integration tests more stable in CI#19995
RobinMalfait merged 2 commits into
mainfrom
fix/improve-ci-integrations

Conversation

@RobinMalfait
Copy link
Copy Markdown
Member

@RobinMalfait RobinMalfait commented Apr 29, 2026

This PR is an attempt to make CI more stable related to integration tests.

[ci-all]

@RobinMalfait RobinMalfait force-pushed the fix/improve-ci-integrations branch from d747f98 to aee1a8b Compare April 30, 2026 20:36
@RobinMalfait RobinMalfait marked this pull request as ready for review April 30, 2026 21:23
@RobinMalfait RobinMalfait requested a review from a team as a code owner April 30, 2026 21:23
@RobinMalfait RobinMalfait merged commit 0db856f into main Apr 30, 2026
23 checks passed
@RobinMalfait RobinMalfait deleted the fix/improve-ci-integrations branch April 30, 2026 21:23
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Apr 30, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 194967d0-9ed5-4304-80b3-ab8adf492840

📥 Commits

Reviewing files that changed from the base of the PR and between 1ca0aac and aee1a8b.

📒 Files selected for processing (2)
  • integrations/utils.ts
  • integrations/vite/nuxt.test.ts

Walkthrough

This pull request modifies the test utilities and integration test infrastructure. The test() wrapper in integrations/utils.ts now accepts a concurrent flag that is forwarded to Vitest instead of forcing concurrent execution by default, and a new test.concurrent convenience function is added. Spawned-process cleanup is enhanced with process detachment for non-Windows systems, asynchronous disposal using a process-tree kill strategy (SIGTERM followed by SIGKILL or Windows taskkill), and timeout-based forced termination. Cleanup outcomes are aggregated with Promise.allSettled, throwing an AggregateError if any disposals fail. A new getRandomPort() function allocates available local TCP ports. The test file is updated to use dynamic port allocation instead of a hardcoded port value.

Review rate limit: 3/5 reviews remaining, refill in 19 minutes and 24 seconds.

Comment @coderabbitai help to get the list of available commands and usage tips.

iamseopyeongwon pushed a commit to iamseopyeongwon/tailwindcss that referenced this pull request May 10, 2026
@depfu
Update eslint 9.32.0 → 9.33.0 (minor) (tailwindlabs#18779)
e4c0255 
Here is everything you need to know about this update. Please take a
good look at what changed and the test results before merging this pull
request.

### What changed?




#### ✳️ eslint (9.32.0 → 9.33.0) ·
[Repo](https://github.com/eslint/eslint) ·
[Changelog](https://github.com/eslint/eslint/blob/main/CHANGELOG.md)



<details>
<summary>Release Notes</summary>
<h4><a
href="https://github.com/eslint/eslint/releases/tag/v9.33.0">9.33.0</a></h4>

<blockquote><h2 dir="auto">Features</h2>
<ul dir="auto">
<li>
<a
href="https://bounce.depfu.com/github.com/eslint/eslint/commit/e07820e66fd1fceaf2620dc931154955a706cc0f"><code
class="notranslate">e07820e</code></a> feat: add global object access
detection to no-restricted-globals (<a
href="https://bounce.depfu.com/github.com/eslint/eslint/pull/19939">#19939</a>)
(sethamus)</li>
<li>
<a
href="https://bounce.depfu.com/github.com/eslint/eslint/commit/90b050ec11557cab08b6be9f05fabf97dba6a63d"><code
class="notranslate">90b050e</code></a> feat: support explicit resource
management in <code class="notranslate">one-var</code> (<a
href="https://bounce.depfu.com/github.com/eslint/eslint/pull/19941">#19941</a>)
(Sweta Tanwar)</li>
</ul>
<h2 dir="auto">Bug Fixes</h2>
<ul dir="auto">
<li>
<a
href="https://bounce.depfu.com/github.com/eslint/eslint/commit/732433c4fb023f45154b825cdc8cdaf1979d4336"><code
class="notranslate">732433c</code></a> fix: allow any type for <code
class="notranslate">meta.docs.recommended</code> in custom rules (<a
href="https://bounce.depfu.com/github.com/eslint/eslint/pull/19995">#19995</a>)
(Francesco Trotta)</li>
<li>
<a
href="https://bounce.depfu.com/github.com/eslint/eslint/commit/e8a6914a249d036e12494004e586b2a2b6e104d1"><code
class="notranslate">e8a6914</code></a> fix: Fixed potential bug in
check-emfile-handling.js (<a
href="https://bounce.depfu.com/github.com/eslint/eslint/pull/19975">#19975</a>)
(諏訪原慶斗)</li>
</ul>
<h2 dir="auto">Documentation</h2>
<ul dir="auto">
<li>
<a
href="https://bounce.depfu.com/github.com/eslint/eslint/commit/34f0723e2d0faf8ac8dc95ec56e6d181bd6b67f2"><code
class="notranslate">34f0723</code></a> docs: playground button for
TypeScript code example (<a
href="https://bounce.depfu.com/github.com/eslint/eslint/pull/19671">#19671</a>)
(Tanuj Kanti)</li>
<li>
<a
href="https://bounce.depfu.com/github.com/eslint/eslint/commit/dc942a47daf41228d69072c52f1be20789426862"><code
class="notranslate">dc942a4</code></a> docs: Update README (GitHub
Actions Bot)</li>
<li>
<a
href="https://bounce.depfu.com/github.com/eslint/eslint/commit/5a4b6f74320b72f9b6ad8b30f5c463b2b71315af"><code
class="notranslate">5a4b6f7</code></a> docs: Update no-multi-assign.md
(<a
href="https://bounce.depfu.com/github.com/eslint/eslint/pull/19979">#19979</a>)
(Yuki Takada (Yukinosuke Takada))</li>
<li>
<a
href="https://bounce.depfu.com/github.com/eslint/eslint/commit/247e15698e34919a0cd411842fb3e14ac7a8f1ba"><code
class="notranslate">247e156</code></a> docs: add missing let
declarations in <code class="notranslate">no-plusplus</code> (<a
href="https://bounce.depfu.com/github.com/eslint/eslint/pull/19980">#19980</a>)
(Yuki Takada (Yukinosuke Takada))</li>
<li>
<a
href="https://bounce.depfu.com/github.com/eslint/eslint/commit/0d17242b3c25c2ddf8363f4560641acd1ae82ca9"><code
class="notranslate">0d17242</code></a> docs: Update README (GitHub
Actions Bot)</li>
<li>
<a
href="https://bounce.depfu.com/github.com/eslint/eslint/commit/fa20b9db8ff90ea9f0527118114dda17c656d095"><code
class="notranslate">fa20b9d</code></a> docs: Clarify when to open an
issue for a PR (<a
href="https://bounce.depfu.com/github.com/eslint/eslint/pull/19974">#19974</a>)
(Nicholas C. Zakas)</li>
</ul>
<h2 dir="auto">Build Related</h2>
<ul dir="auto">
<li>
<a
href="https://bounce.depfu.com/github.com/eslint/eslint/commit/27fa86551bd173387e29a139293de78b0e14f0f3"><code
class="notranslate">27fa865</code></a> build: use <code
class="notranslate">ESLint</code> class to generate formatter examples
(<a
href="https://bounce.depfu.com/github.com/eslint/eslint/pull/19972">#19972</a>)
(Milos Djermanovic)</li>
</ul>
<h2 dir="auto">Chores</h2>
<ul dir="auto">
<li>
<a
href="https://bounce.depfu.com/github.com/eslint/eslint/commit/425804602ecb9ee5f54d1c38a473cf20538420c5"><code
class="notranslate">4258046</code></a> chore: update dependency
@eslint/js to v9.33.0 (<a
href="https://bounce.depfu.com/github.com/eslint/eslint/pull/19998">#19998</a>)
(renovate[bot])</li>
<li>
<a
href="https://bounce.depfu.com/github.com/eslint/eslint/commit/ad283717ed4764a171120ca7c6cba82a78fa024c"><code
class="notranslate">ad28371</code></a> chore: package.json update for
@eslint/js release (Jenkins)</li>
<li>
<a
href="https://bounce.depfu.com/github.com/eslint/eslint/commit/06a22f154c08ea044b3172b357b226d34dfefc6a"><code
class="notranslate">06a22f1</code></a> test: resolve flakiness in --mcp
flag test (<a
href="https://bounce.depfu.com/github.com/eslint/eslint/pull/19993">#19993</a>)
(Pixel998)</li>
<li>
<a
href="https://bounce.depfu.com/github.com/eslint/eslint/commit/54920ed229693f23650dace6e567bf44413aaf98"><code
class="notranslate">54920ed</code></a> test: switch to <code
class="notranslate">Linter.Config</code> in <code
class="notranslate">ESLintRules</code> type tests (<a
href="https://bounce.depfu.com/github.com/eslint/eslint/pull/19977">#19977</a>)
(Francesco Trotta)</li>
</ul></blockquote>
<p><em>Does any of this look wrong? <a
href="https://depfu.com/packages/npm/eslint/feedback">Please let us
know.</a></em></p>
</details>

<details>
<summary>Commits</summary>
<p><a
href="https://github.com/eslint/eslint/compare/2364031090daafe34e80904c32065bfe4692d7a2...a90d7c4fe5ef83054e29d21d7ffb442103429d03">See
the full diff on Github</a>. The new version differs by 17 commits:</p>
<ul>
<li><a
href="https://github.com/eslint/eslint/commit/a90d7c4fe5ef83054e29d21d7ffb442103429d03"><code>9.33.0</code></a></li>
<li><a
href="https://github.com/eslint/eslint/commit/9534b55372dbff3bf153384ed2aaab172f998175"><code>Build:
changelog update for 9.33.0</code></a></li>
<li><a
href="https://github.com/eslint/eslint/commit/425804602ecb9ee5f54d1c38a473cf20538420c5"><code>chore:
update dependency @eslint/js to v9.33.0 (tailwindlabs#19998)</code></a></li>
<li><a
href="https://github.com/eslint/eslint/commit/ad283717ed4764a171120ca7c6cba82a78fa024c"><code>chore:
package.json update for @eslint/js release</code></a></li>
<li><a
href="https://github.com/eslint/eslint/commit/06a22f154c08ea044b3172b357b226d34dfefc6a"><code>test:
resolve flakiness in --mcp flag test (tailwindlabs#19993)</code></a></li>
<li><a
href="https://github.com/eslint/eslint/commit/732433c4fb023f45154b825cdc8cdaf1979d4336"><code>fix:
allow any type for `meta.docs.recommended` in custom rules
(tailwindlabs#19995)</code></a></li>
<li><a
href="https://github.com/eslint/eslint/commit/34f0723e2d0faf8ac8dc95ec56e6d181bd6b67f2"><code>docs:
playground button for TypeScript code example (tailwindlabs#19671)</code></a></li>
<li><a
href="https://github.com/eslint/eslint/commit/dc942a47daf41228d69072c52f1be20789426862"><code>docs:
Update README</code></a></li>
<li><a
href="https://github.com/eslint/eslint/commit/5a4b6f74320b72f9b6ad8b30f5c463b2b71315af"><code>docs:
Update no-multi-assign.md (tailwindlabs#19979)</code></a></li>
<li><a
href="https://github.com/eslint/eslint/commit/247e15698e34919a0cd411842fb3e14ac7a8f1ba"><code>docs:
add missing let declarations in `no-plusplus` (tailwindlabs#19980)</code></a></li>
<li><a
href="https://github.com/eslint/eslint/commit/0d17242b3c25c2ddf8363f4560641acd1ae82ca9"><code>docs:
Update README</code></a></li>
<li><a
href="https://github.com/eslint/eslint/commit/e07820e66fd1fceaf2620dc931154955a706cc0f"><code>feat:
add global object access detection to no-restricted-globals
(tailwindlabs#19939)</code></a></li>
<li><a
href="https://github.com/eslint/eslint/commit/fa20b9db8ff90ea9f0527118114dda17c656d095"><code>docs:
Clarify when to open an issue for a PR (tailwindlabs#19974)</code></a></li>
<li><a
href="https://github.com/eslint/eslint/commit/54920ed229693f23650dace6e567bf44413aaf98"><code>test:
switch to `Linter.Config` in `ESLintRules` type tests
(tailwindlabs#19977)</code></a></li>
<li><a
href="https://github.com/eslint/eslint/commit/e8a6914a249d036e12494004e586b2a2b6e104d1"><code>fix:
Fixed potential bug in check-emfile-handling.js (tailwindlabs#19975)</code></a></li>
<li><a
href="https://github.com/eslint/eslint/commit/90b050ec11557cab08b6be9f05fabf97dba6a63d"><code>feat:
support explicit resource management in `one-var`
(tailwindlabs#19941)</code></a></li>
<li><a
href="https://github.com/eslint/eslint/commit/27fa86551bd173387e29a139293de78b0e14f0f3"><code>build:
use `ESLint` class to generate formatter examples
(tailwindlabs#19972)</code></a></li>
</ul>
</details>












---
![Depfu
Status](https://depfu.com/badges/edd6acd35d74c8d41cbb540c30442adf/stats.svg)

[Depfu](https://depfu.com) will automatically keep this PR
conflict-free, as long as you don't add any commits to this branch
yourself. You can also trigger a rebase manually by commenting with
`@depfu rebase`.

<details><summary>All Depfu comment commands</summary>
<blockquote><dl>
<dt>@​depfu rebase</dt><dd>Rebases against your default branch and
redoes this update</dd>
<dt>@​depfu recreate</dt><dd>Recreates this PR, overwriting any edits
that you've made to it</dd>
<dt>@​depfu merge</dt><dd>Merges this PR once your tests are passing and
conflicts are resolved</dd>
<dt>@​depfu cancel merge</dt><dd>Cancels automatic merging of this
PR</dd>
<dt>@​depfu close</dt><dd>Closes this PR and deletes the branch</dd>
<dt>@​depfu reopen</dt><dd>Restores the branch and reopens this PR (if
it's closed)</dd>
<dt>@​depfu pause</dt><dd>Ignores all future updates for this dependency
and closes this PR</dd>
<dt>@​depfu pause [minor|major]</dt><dd>Ignores all future minor/major
updates for this dependency and closes this PR</dd>
<dt>@​depfu resume</dt><dd>Future versions of this dependency will
create PRs again (leaves this PR as is)</dd>
</dl></blockquote>
</details>

Co-authored-by: depfu[bot] <23717796+depfu[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@RobinMalfait