Skip to content

ci: set hook-timeout on CI#1113

Merged
colinaaa merged 1 commit intolynx-family:mainfrom
colinaaa:colin/0619/hook-timeout
Jun 19, 2025
Merged

ci: set hook-timeout on CI#1113
colinaaa merged 1 commit intolynx-family:mainfrom
colinaaa:colin/0619/hook-timeout

Conversation

@colinaaa
Copy link
Copy Markdown
Collaborator

Summary

Fix Windows CI failure. See: https://github.com/lynx-family/lynx-stack/actions/runs/15757466877/job/44416369201?pr=1111

close: m-6701643515

Checklist

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

@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented Jun 19, 2025

⚠️ No Changeset found

Latest commit: 6ab0694

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

@codecov
Copy link
Copy Markdown

codecov Bot commented Jun 19, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

✅ All tests successful. No failed tests found.

📢 Thoughts on this report? Let us know!

@codspeed-hq
Copy link
Copy Markdown

codspeed-hq Bot commented Jun 19, 2025

CodSpeed Performance Report

Merging #1113 will not alter performance

Comparing colinaaa:colin/0619/hook-timeout (6ab0694) with main (9513bbf)

Summary

✅ 8 untouched benchmarks

@relativeci
Copy link
Copy Markdown

relativeci Bot commented Jun 19, 2025

Web Explorer

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

6ab0694(current) vs 9513bbf main#1766(baseline)

Bundle metrics  Change 2 changes
                 Current
#1776
     Baseline
#1766
No change  Initial JS 140.06KiB 140.06KiB
No change  Initial CSS 31.82KiB 31.82KiB
Change  Cache Invalidation 0% 54.29%
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
#1776
     Baseline
#1766
No change  JS 226KiB 226KiB
No change  CSS 31.82KiB 31.82KiB

Bundle analysis reportBranch colinaaa:colin/0619/hook-timeoutProject dashboard


Generated by RelativeCIDocumentationReport issue

@relativeci
Copy link
Copy Markdown

relativeci Bot commented Jun 19, 2025

React Example

#1787 Bundle Size — 232.61KiB (0%).

6ab0694(current) vs 9513bbf main#1777(baseline)

Bundle metrics  no changes
                 Current
#1787
     Baseline
#1777
No change  Initial JS 0B 0B
No change  Initial CSS 0B 0B
No change  Cache Invalidation 0% 0%
No change  Chunks 0 0
No change  Assets 4 4
No change  Modules 143 143
No change  Duplicate Modules 55 55
No change  Duplicate Code 45.68% 45.68%
No change  Packages 2 2
No change  Duplicate Packages 0 0
Bundle size by type  no changes
                 Current
#1787
     Baseline
#1777
No change  IMG 145.76KiB 145.76KiB
No change  Other 86.86KiB 86.86KiB

Bundle analysis reportBranch colinaaa:colin/0619/hook-timeoutProject dashboard


Generated by RelativeCIDocumentationReport issue

@colinaaa colinaaa marked this pull request as ready for review June 19, 2025 12:43
@colinaaa colinaaa merged commit 3bbae79 into lynx-family:main Jun 19, 2025
38 of 41 checks passed
@colinaaa colinaaa deleted the colin/0619/hook-timeout branch June 19, 2025 12:43
colinaaa added a commit that referenced this pull request Jun 20, 2025
## 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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant