Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove deprecated functions delay, repeat, loop, sequence #3480

Merged
merged 1 commit into from
Aug 22, 2022

Conversation

tomekzaw
Copy link
Member

@tomekzaw tomekzaw commented Aug 17, 2022

Description

This PR removes the following functions which were deprecated in 2.x:

Changes

  • Removed deprecated functions delay, repeat, loop, sequence

Test code and steps to reproduce

Checklist

  • Included code example that can be used to test this change
  • Updated TS types
  • Added TS types tests
  • Added unit / integration tests
  • Updated documentation
  • Ensured that CI passes

@tomekzaw tomekzaw changed the title Remove deprecated functions delay, repeat, loop, sequence [v3] Remove deprecated functions delay, repeat, loop, sequence Aug 17, 2022
@tomekzaw tomekzaw changed the title [v3] Remove deprecated functions delay, repeat, loop, sequence Remove deprecated functions delay, repeat, loop, sequence Aug 17, 2022
Copy link
Member

@piaskowyk piaskowyk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@piaskowyk piaskowyk merged commit 1f2e428 into main Aug 22, 2022
@piaskowyk piaskowyk deleted the @tomekzaw/remove-deprecated-functions branch August 22, 2022 12:52
fluiddot pushed a commit to wordpress-mobile/react-native-reanimated that referenced this pull request Jun 5, 2023
## Description

This PR removes the following functions which were deprecated in 2.x:
* `delay` → use [`withDelay`](https://docs.swmansion.com/react-native-reanimated/docs/api/animations/withDelay/) instead
* `repeat` → use [`withRepeat`](https://docs.swmansion.com/react-native-reanimated/docs/api/animations/withRepeat/) instead
* `loop` → use [`withRepeat`](https://docs.swmansion.com/react-native-reanimated/docs/api/animations/withRepeat/) instead
* `sequence` → use [`withSequence`](https://docs.swmansion.com/react-native-reanimated/docs/api/animations/withSequence/) instead

<!--
Description and motivation for this PR.

Inlude Fixes #<number> if this is fixing some issue.

Fixes # .
-->

## Changes

- Removed deprecated functions `delay`, `repeat`, `loop`, `sequence`

<!--
Please describe things you've changed here, make a **high level** overview, if change is simple you can omit this section.

For example:

- Added `foo` method which add bouncing animation
- Updated `about.md` docs
- Added caching in CI builds

-->

<!--

## Screenshots / GIFs

Here you can add screenshots / GIFs documenting your change.

You can add before / after section if you're changing some behavior.

### Before

### After

-->

## Test code and steps to reproduce

<!--
Please include code that can be used to test this change and short description how this example should work.
This snippet should be as minimal as possible and ready to be pasted into editor (don't exclude exports or remove "not important" parts of reproduction example)
-->

## Checklist

- [ ] Included code example that can be used to test this change
- [ ] Updated TS types
- [ ] Added TS types tests
- [ ] Added unit / integration tests
- [ ] Updated documentation
- [ ] Ensured that CI passes
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants