Skip to content

Conversation

@MShekow
Copy link

@MShekow MShekow commented Aug 2, 2016

Added order parameter to 2.0 changelog. Added check_params_cb parameter documentation to waitSignals. Added check_params_cbs and order parameter documentation to waitSignals.

…er documentation to waitSignals. Added check_params_cbs and order parameter documentation to waitSignals. #147
docs/signals.rst Outdated
.. versionadded:: 2.0

If the signal has parameters you want to compare with expected values, you can pass
``check_params_cb=some_callable(*parameters)`` that compares the provided signal parameters to some expected parameters.
Copy link
Member

Choose a reason for hiding this comment

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

The (*parameters) seems confusing to me here - why not leave it off?

@The-Compiler
Copy link
Member

Great docs! Note that the title underlining should be the same length as the title, otherwise rst-lint will fail. I also added some minor comments.

@coveralls
Copy link

coveralls commented Aug 2, 2016

Coverage Status

Coverage remained the same at 100.0% when pulling 474e934 on MShekow:master into eabecd1 on pytest-dev:master.

@coveralls
Copy link

coveralls commented Aug 2, 2016

Coverage Status

Coverage remained the same at 100.0% when pulling 07c0efe on MShekow:master into eabecd1 on pytest-dev:master.

@nicoddemus nicoddemus merged commit e5f4467 into pytest-dev:master Aug 2, 2016
@nicoddemus
Copy link
Member

Thanks a lot! 😄

@MShekow
Copy link
Author

MShekow commented Aug 2, 2016

No problem ;)

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.

4 participants