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

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

5 participants