Skip to content

Allow signal arguments in contextmanager emitting#16

Open
monkpit wants to merge 1 commit intoshaunduncan:masterfrom
monkpit:patch-1
Open

Allow signal arguments in contextmanager emitting#16
monkpit wants to merge 1 commit intoshaunduncan:masterfrom
monkpit:patch-1

Conversation

@monkpit
Copy link
Copy Markdown

@monkpit monkpit commented May 20, 2016

Allow contextmanager emitting to pass additional arguments to emit.

Allow contextmanager `emitting` to pass additional arguments to `emit`.
@monkpit
Copy link
Copy Markdown
Author

monkpit commented May 20, 2016

I added some simple *args, **kwargs to the arguments of emitting because I wanted to do this:

with emitting(enter='on_start', exit='on_end', foo=1, bar=2):
    # do stuff

Another way to handle this would be to explicitly have an args parameter that can be passed a tuple of positional arguments.

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.

1 participant