Skip to content
This repository has been archived by the owner on Jan 24, 2024. It is now read-only.

[SideEffect] add side effects support for list #220

Merged
merged 17 commits into from
Jul 4, 2023
Merged

Conversation

SigureMo
Copy link
Member

@SigureMo SigureMo commented Jun 29, 2023

一个非常基础的 list 副作用恢复机制

@paddle-bot
Copy link

paddle-bot bot commented Jun 29, 2023

Thanks for your contribution!

@SigureMo SigureMo marked this pull request as ready for review July 4, 2023 02:44
@SigureMo SigureMo changed the title [WIP][SideEffect] add side effects support for list [SideEffect] add side effects support for list Jul 4, 2023
Aurelius84
Aurelius84 previously approved these changes Jul 4, 2023
Copy link
Collaborator

@Aurelius84 Aurelius84 left a comment

Choose a reason for hiding this comment

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

Great job!

Copy link
Collaborator

@2742195759 2742195759 left a comment

Choose a reason for hiding this comment

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

LGTM

@SigureMo SigureMo merged commit 6f03436 into develop Jul 4, 2023
5 checks passed
@SigureMo SigureMo deleted the side-effects-list branch July 4, 2023 09:05
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants