Skip to content

Conversation

@agrawalprash
Copy link
Contributor

No description provided.

@prabhuramachandran
Copy link
Member

@epatters -- can you please review this?

@WarrenWeckesser
Copy link

Needs a unit test.

@epatters
Copy link
Contributor

This looks fine to me.

It would be nice to add a unit test, but I'm not going to require that you do so since ListeningAction has no tests to begin with. However, if you write some tests, you will have my gratitude :)

@dpinte dpinte added this to the 4.6.0 version milestone Mar 13, 2015
@rahulporuri rahulporuri removed this from the 4.6.0 version milestone Jul 13, 2020
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.

6 participants