Skip to content

Convert fastMerge to worklet to be able to use it in animations on NewDot#855

Merged
JS00001 merged 2 commits into
Expensify:mainfrom
parasharrajat:fix/fast-merge-worklet
May 27, 2025
Merged

Convert fastMerge to worklet to be able to use it in animations on NewDot#855
JS00001 merged 2 commits into
Expensify:mainfrom
parasharrajat:fix/fast-merge-worklet

Conversation

@parasharrajat

Copy link
Copy Markdown
Member

Fixed Issues

$ Expensify/App#62862

Tests

  1. What unit/integration tests cover your change? What autoQA tests cover your change?
  2. What tests did you perform that validates your changed worked?

QA

  1. What does QA need to do to validate your changes?
  2. What areas to they need to test for regressions?

@parasharrajat parasharrajat requested a review from a team as a code owner May 27, 2025 17:08
@melvin-bot melvin-bot Bot requested review from MonilBhavsar and removed request for a team May 27, 2025 17:09
@parasharrajat

Copy link
Copy Markdown
Member Author

We can use the top-level worklet; when all the functions are in same file. Source https://expensify.slack.com/archives/C05LX9D6E07/p1748363139288069?thread_ts=1748347517.737619&cid=C05LX9D6E07

@JS00001 JS00001 requested review from JS00001 and removed request for MonilBhavsar May 27, 2025 17:17
@JS00001 JS00001 merged commit 88bbe74 into Expensify:main May 27, 2025
6 checks passed
@os-botify

os-botify Bot commented May 27, 2025

Copy link
Copy Markdown
Contributor

🚀 Published to npm in 2.0.137 🎉

@parasharrajat parasharrajat changed the title Convert fastmerge to worklet to be able to use it in animations on NewDot Convert fastMerge to worklet to be able to use it in animations on NewDot May 27, 2025
@Skalakid Skalakid mentioned this pull request Jul 9, 2025
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