Skip to content

ci: use --testTimeout instead of --test-timeout#1121

Merged
colinaaa merged 1 commit intolynx-family:mainfrom
colinaaa:colin/0620/timeout
Jun 20, 2025
Merged

ci: use --testTimeout instead of --test-timeout#1121
colinaaa merged 1 commit intolynx-family:mainfrom
colinaaa:colin/0620/timeout

Conversation

@colinaaa
Copy link
Copy Markdown
Collaborator

@colinaaa colinaaa commented Jun 20, 2025

Summary

Vitest docs: https://vitest.dev/guide/cli.html#testtimeout

Checklist

  • Tests updated (or not required).
  • Documentation updated (or not required).

@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented Jun 20, 2025

⚠️ No Changeset found

Latest commit: 1d8c886

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@codspeed-hq
Copy link
Copy Markdown

codspeed-hq Bot commented Jun 20, 2025

CodSpeed Performance Report

Merging #1121 will not alter performance

Comparing colinaaa:colin/0620/timeout (1d8c886) with main (6137fba)

Summary

✅ 8 untouched benchmarks

@relativeci
Copy link
Copy Markdown

relativeci Bot commented Jun 20, 2025 •
edited
Loading

React Example

#1859 Bundle Size — 233.47KiB (0%).

1d8c886(current) vs 6137fba main#1849(baseline)

Bundle metrics  no changes
                 Current
#1859		      Baseline
#1849
No change  Initial JS 0B 0B
No change  Initial CSS 0B 0B
Change  Cache Invalidation 0% 37.34%
No change  Chunks 0 0
No change  Assets 4 4
No change  Modules 145 145
No change  Duplicate Modules 56 56
No change  Duplicate Code 45.66% 45.66%
No change  Packages 2 2
No change  Duplicate Packages 0 0
Bundle size by type  no changes
                 Current
#1859		      Baseline
#1849
No change  IMG 145.76KiB 145.76KiB
No change  Other 87.71KiB 87.71KiB

Bundle analysis report Branch colinaaa:colin/0620/timeout Project dashboard

Generated by RelativeCI Documentation Report issue

@relativeci
Copy link
Copy Markdown

relativeci Bot commented Jun 20, 2025 •
edited
Loading

Web Explorer

#1848 Bundle Size — 257.82KiB (0%).

1d8c886(current) vs 6137fba main#1838(baseline)

Bundle metrics  Change 2 changes
                 Current
#1848		      Baseline
#1838
No change  Initial JS 140.05KiB 140.05KiB
No change  Initial CSS 31.82KiB 31.82KiB
No change  Cache Invalidation 0% 0%
No change  Chunks 4 4
No change  Assets 4 4
Change  Modules 205(-0.49%) 206
No change  Duplicate Modules 15 15
Change  Duplicate Code 3.41%(+0.29%) 3.4%
No change  Packages 4 4
No change  Duplicate Packages 0 0
Bundle size by type  no changes
                 Current
#1848		      Baseline
#1838
No change  JS 226KiB 226KiB
No change  CSS 31.82KiB 31.82KiB

Bundle analysis report Branch colinaaa:colin/0620/timeout Project dashboard

Generated by RelativeCI Documentation Report issue

@colinaaa colinaaa marked this pull request as ready for review June 20, 2025 09:23
@colinaaa colinaaa added this pull request to the merge queue Jun 20, 2025
Merged via the queue into lynx-family:main with commit 3194c3b Jun 20, 2025
63 of 65 checks passed
@colinaaa colinaaa deleted the colin/0620/timeout branch June 20, 2025 11:24
@colinaaa colinaaa mentioned this pull request Jun 20, 2025
Merged
2 tasks
colinaaa added a commit that referenced this pull request Jun 20, 2025
@colinaaa
test(tailwind-preset): avoid building tailwindcss in test (#1125)
6fbe965 
## Summary

<!-- Can you explain the reasoning behind implementing this change? What
problem or issue does this pull request resolve? -->

The `beforeAll` hook of
`tools/tailwind-preset/src/__tests__/config.test.ts` is failing all the
time on Windows.

I'm not sure why does setting `--hookTimeout` not working (#1113,
##1121).

```
 FAIL  
tools/tailwind-preset src/__tests__/config.test.ts > Lynx Tailwind Preset
Error: Hook timed out in 10000ms.
If this is a long-running hook, pass a timeout value as the last argument or configure it globally with "hookTimeout".
```

<!-- It would be helpful if you could provide any relevant context, such
as GitHub issues or related discussions. -->

Also, the `__tests__` and `.js.map` are excluded from
`package.json#files` to reduce package size.

## Checklist

<!--- Check and mark with an "x" -->

- [x] Tests updated (or not required).
- [ ] Documentation updated (or not required).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Yradex Yradex Yradex approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@colinaaa @Yradex