Skip to content

fix: avoid override user defined fetch function#181

Closed
upupming wants to merge 1 commit intolynx-family:mainfrom
upupming:fix/fetch
Closed

fix: avoid override user defined fetch function#181
upupming wants to merge 1 commit intolynx-family:mainfrom
upupming:fix/fetch

Conversation

@upupming
Copy link
Copy Markdown
Collaborator

Summary

Checklist

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

@upupming upupming requested a review from colinaaa as a code owner March 12, 2025 05:49
@changeset-bot
Copy link
Copy Markdown

changeset-bot bot commented Mar 12, 2025

🦋 Changeset detected

Latest commit: 4cbaa81

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

This PR includes changesets to release 3 packages
Name Type
@lynx-js/runtime-wrapper-webpack-plugin Patch
@lynx-js/react-rsbuild-plugin Patch
@lynx-js/react-alias-rsbuild-plugin Patch

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

@codecov
Copy link
Copy Markdown

codecov bot commented Mar 12, 2025

Codecov Report

Attention: Patch coverage is 90.90909% with 1 line in your changes missing coverage. Please review.

✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
...-webpack-plugin/src/RuntimeWrapperWebpackPlugin.ts 87.50% 1 Missing ⚠️

📢 Thoughts on this report? Let us know!

@upupming upupming marked this pull request as draft March 12, 2025 06:28
@upupming upupming force-pushed the fix/fetch branch 3 times, most recently from 9ce38be to 0978ef3 Compare March 18, 2025 06:55
@upupming upupming marked this pull request as ready for review March 18, 2025 07:57
@upupming upupming requested review from gaoachao and hzy March 18, 2025 07:57
gaoachao
gaoachao previously approved these changes Mar 18, 2025
@upupming upupming force-pushed the fix/fetch branch 2 times, most recently from d874ed2 to 4cbaa81 Compare March 18, 2025 09:11
github-merge-queue bot pushed a commit that referenced this pull request May 13, 2025
## Summary

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

Allow direct access to `requestAnimationFrame` and
`cancelAnimationFrame` in background thread script.

close: #181 since `fetch` is now a parameter of the runtime wrapper.

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

## 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.

2 participants