Skip to content

Conversation

@rjrjr
Copy link
Collaborator

@rjrjr rjrjr commented Jun 29, 2020

A tiny bit of convenience, a lot of landmines.

@rjrjr rjrjr requested a review from a team as a code owner June 29, 2020 23:27
@rjrjr
Copy link
Collaborator Author

rjrjr commented Jun 29, 2020

Oops, we're using emitOutput in a test.

@rjrjr rjrjr force-pushed the rjrjr/no-enterState branch from ce6242f to 40b7e47 Compare June 29, 2020 23:47
A tiny bit of convenience, a lot of landmines.
@rjrjr rjrjr force-pushed the rjrjr/no-enterState branch from 40b7e47 to 4a27e76 Compare June 30, 2020 00:25
Copy link
Collaborator

@zach-klippenstein zach-klippenstein left a comment

Choose a reason for hiding this comment

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

LG, but I'm a bit worried this is going to add a lot of unnecessary overhead to upgrading internally. Yes these have been deprecated for a while, and yes they're bad, but they're no more bad now than they were last month.

@rjrjr
Copy link
Collaborator Author

rjrjr commented Jun 30, 2020

I'll close it then.

@rjrjr rjrjr closed this Jun 30, 2020
zach-klippenstein added a commit that referenced this pull request Feb 4, 2021
Move everything into a compose subdirectory to prep for importing into the main workflow-kotlin repo.
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.

3 participants