Skip to content

Conversation

@colinaaa
Copy link
Collaborator

@colinaaa colinaaa commented Jun 5, 2025

Summary

Similar to #1007, we normalize the relative path to ensure consistent snapshot generation across Windows, Linux, and macOS.

Note

The testing library does not work on Windows now since we are using environment with a absolute path.
An error "Cannot find dependency 'vitest-environment-C:\Users\Admin\colin\lynx-stack\packages\react\testing-library\dist\env\vitest.js'" would occur.

I've submitted a fix at vitest-dev/vitest#8105

issue: #81

Checklist

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

@changeset-bot
Copy link

changeset-bot bot commented Jun 5, 2025

🦋 Changeset detected

Latest commit: 0f54973

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 0 packages

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

Not sure what this means? Click here to learn what changesets are.

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

@codspeed-hq
Copy link

codspeed-hq bot commented Jun 5, 2025

CodSpeed Performance Report

Merging #1011 will not alter performance

Comparing colinaaa:colin/0605/windows-test-3 (0f54973) with main (5be9cbd)

Summary

✅ 8 untouched benchmarks

@colinaaa colinaaa marked this pull request as ready for review June 5, 2025 12:57
@colinaaa colinaaa requested a review from upupming June 5, 2025 12:57
@relativeci
Copy link

relativeci bot commented Jun 5, 2025

React Example

#1273 Bundle Size — 231.7KiB (0%).

0f54973(current) vs 5be9cbd main#1266(baseline)

Bundle metrics  no changes
                 Current
#1273
     Baseline
#1266
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 142 142
No change  Duplicate Modules 56 56
No change  Duplicate Code 46.16% 46.16%
No change  Packages 2 2
No change  Duplicate Packages 0 0
Bundle size by type  no changes
                 Current
#1273
     Baseline
#1266
No change  IMG 145.76KiB 145.76KiB
No change  Other 85.94KiB 85.94KiB

Bundle analysis reportBranch colinaaa:colin/0605/windows-test...Project dashboard


Generated by RelativeCIDocumentationReport issue

@relativeci
Copy link

relativeci bot commented Jun 5, 2025

Web Explorer

#1262 Bundle Size — 254.27KiB (0%).

0f54973(current) vs 5be9cbd main#1255(baseline)

Bundle metrics  Change 2 changes
                 Current
#1262
     Baseline
#1255
No change  Initial JS 137.96KiB 137.96KiB
No change  Initial CSS 31.79KiB 31.79KiB
No change  Cache Invalidation 0% 0%
No change  Chunks 4 4
No change  Assets 4 4
Change  Modules 201(-0.5%) 202
No change  Duplicate Modules 14 14
Change  Duplicate Code 2.87%(+0.35%) 2.86%
No change  Packages 4 4
No change  Duplicate Packages 0 0
Bundle size by type  no changes
                 Current
#1262
     Baseline
#1255
No change  JS 222.48KiB 222.48KiB
No change  CSS 31.79KiB 31.79KiB

Bundle analysis reportBranch colinaaa:colin/0605/windows-test...Project dashboard


Generated by RelativeCIDocumentationReport issue

@upupming upupming added this pull request to the merge queue Jun 6, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to no response for status checks Jun 6, 2025
@upupming upupming added this pull request to the merge queue Jun 6, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to no response for status checks Jun 6, 2025
@colinaaa colinaaa added this pull request to the merge queue Jun 6, 2025
Merged via the queue into lynx-family:main with commit 4a69714 Jun 6, 2025
60 of 62 checks passed
@colinaaa colinaaa deleted the colin/0605/windows-test-3 branch June 6, 2025 07:00
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.

2 participants