-
Notifications
You must be signed in to change notification settings - Fork 28
Added step argument transformation for human readable integer expression #66
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, @thathi. Looks good - just a few small changes needed 🙂
bindings/src/Capgemini.PowerApps.SpecFlowBindings/Steps/EntitySubGridSteps.cs
Outdated
Show resolved
Hide resolved
bindings/src/Capgemini.PowerApps.SpecFlowBindings/Steps/GridSteps.cs
Outdated
Show resolved
Hide resolved
bindings/src/Capgemini.PowerApps.SpecFlowBindings/Steps/LookupSteps.cs
Outdated
Show resolved
Hide resolved
bindings/src/Capgemini.PowerApps.SpecFlowBindings/Steps/RelatedGridSteps.cs
Outdated
Show resolved
Hide resolved
...rc/Capgemini.PowerApps.SpecFlowBindings/Steps/StepArgument/HumanReadableIntegerExpression.cs
Outdated
Show resolved
Hide resolved
bindings/src/Capgemini.PowerApps.SpecFlowBindings/Transformations/UtilityTransformations.cs
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, @thathi! I'll just run the validation pipeline and we can get this merged.
/azp run Capgemini.PowerApps.SpecFlowBindings PR |
Azure Pipelines successfully started running 1 pipeline(s). |
/azp run Capgemini.PowerApps.SpecFlowBindings PR |
Commenter does not have sufficient privileges for PR 66 in repo Capgemini/powerapps-specflow-bindings |
@thathi Don't worry - it didn't work for me either! 😄 I had to manually start it in Azure DevOps. |
Kudos, SonarCloud Quality Gate passed! |
Purpose
#65 Readability
Approach
And I open the record at position 1st in the grid
TODOs