You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I started using behave-django today and ran fairly quickly into the issue that django_ready() hook is documented as taking a 'scenario' argument, but doesn't actually.
Can I suggest that you just update the documentation to correctly reflect the state of the project? While having scenario as an extra option might be useful, you should probably only make a change like that on a major version, or by adding a new additional hook function like django_scenario_ready(). In the meantime, the current version should have accurate documentation, especially for something people are likely to copy and paste in the first time they use the project (like I just did).
The text was updated successfully, but these errors were encountered:
That one has popped up a few times since it was introduced with #46. Should be fixed now with commit 4d8df68. Sorry for the inconvenience. Not sure why this happened since not even the tests show the second argument.
Can you check the updated documentation and close this issue if the change is fine?
This is fixed in the current latest docs (stable is still showing the second argument in addition) and will be rolled out to stable with our next release to PyPI. 🏁
I started using behave-django today and ran fairly quickly into the issue that django_ready() hook is documented as taking a 'scenario' argument, but doesn't actually.
Can I suggest that you just update the documentation to correctly reflect the state of the project? While having scenario as an extra option might be useful, you should probably only make a change like that on a major version, or by adding a new additional hook function like django_scenario_ready(). In the meantime, the current version should have accurate documentation, especially for something people are likely to copy and paste in the first time they use the project (like I just did).
The text was updated successfully, but these errors were encountered: