-
Notifications
You must be signed in to change notification settings - Fork 53
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add Reqnroll.Assist.Dynamic #377
base: main
Are you sure you want to change the base?
Conversation
Does anything speaks against including this to Reqnroll core instead of maintaining it as a separate plugin? |
I also have a few tests that depend on the old Specflow package for this. @gasparnagy don't think so. Would be nice to put it Reqnroll.Assist directly. |
@t0bbz0n @gasparnagy.
|
@olegKoshmeliuk thx. makes sense. then let's keep it as a plugin for now, we can reconsider this later as well. |
@gasparnagy kindly reminder about feedback :) |
🤔 What's changed?
Added ported version of SpecFlow.Assist.Dynamic
⚡️ What's your motivation?
Few times user requested this lib to be as part of Reqnroll.
🏷️ What kind of change is this?
♻️ Anything particular you want feedback on?
Suggestion about better documentation? Technicaly it's library and argument transformation and not a plugin.
Currently this libary tests are not running as part of CI. Better way to implement it?
copyright, credits, authors fields in Reqnroll.Assist.Dynamic.nuspec file. because i just ported library with few changes.
📋 Checklist:
This text was originally taken from the template of the Cucumber project, then edited by hand. You can modify the template here.