Fully removing anonymous action{}
and eventHandler{}
APIs.
#1236
steve-the-edwards
started this conversation in
Polls
Replies: 1 comment
-
Doing this here: #1237 |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
In hindsight Workflow should have never allowed anonymous action {} and eventHandler {}. It makes it too hard to debug. We have now deprecated those versions, forcing callers to provide a name for the action.
Why? Actions are what cause a render pass to happen, so having a name for them is critical to facilitate debugging the cause of a render pass - whether this be for tracing breadcrumbs or monitoring performance in a Perfetto trace.
We would like to follow up with removing them. If you answer yes below, please post as to why and we can talk about ways to mitigate this impact.
0 votes ·
Beta Was this translation helpful? Give feedback.
All reactions