Skip to content

fix(testing-library): rollup dts files#615

Merged
upupming merged 6 commits intolynx-family:mainfrom
upupming:fix/dts-rollup
Apr 24, 2025
Merged

fix(testing-library): rollup dts files#615
upupming merged 6 commits intolynx-family:mainfrom
upupming:fix/dts-rollup

Conversation

@upupming
Copy link
Collaborator

Summary

Fix the type issue of testing library, we should bundle the dts file since we have bundled @lynx-js/test-environment package.

Checklist

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

@changeset-bot
Copy link

changeset-bot bot commented Apr 24, 2025

🦋 Changeset detected

Latest commit: c69032c

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 Apr 24, 2025

CodSpeed Performance Report

Merging #615 will not alter performance

Comparing upupming:fix/dts-rollup (c69032c) with main (a23c578)

Summary

✅ 1 untouched benchmarks

@upupming upupming force-pushed the fix/dts-rollup branch 2 times, most recently from d3a0424 to e32b3b8 Compare April 24, 2025 05:19
@codecov
Copy link

codecov bot commented Apr 24, 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!

colinaaa
colinaaa previously approved these changes Apr 24, 2025
@upupming upupming added this pull request to the merge queue Apr 24, 2025
Merged via the queue into lynx-family:main with commit 7478569 Apr 24, 2025
28 checks passed
@upupming upupming deleted the fix/dts-rollup branch April 24, 2025 08:34
colinaaa added a commit to colinaaa/lynx-stack that referenced this pull request Apr 24, 2025
<!--
  Thank you for submitting a pull request!

We appreciate the time and effort you have invested in making these
changes. Please ensure that you provide enough information to allow
others to review your pull request.

Upon submission, your pull request will be automatically assigned with
reviewers.

If you want to learn more about contributing to this project, please
visit:
https://github.com/lynx-family/lynx-stack/blob/main/CONTRIBUTING.md.
-->

## Summary

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

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

Fix the type issue of testing library, we should bundle the dts file
since we have bundled `@lynx-js/test-environment` package.

## Checklist

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

- [ ] Tests updated (or **not required**).
- [ ] Documentation updated (or **not required**).

---------

Co-authored-by: Qingyu Wang <40660121+colinaaa@users.noreply.github.com>
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