Skip to content
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 RenameCommand #39

Merged
merged 2 commits into from
Jan 21, 2019
Merged

Conversation

slig3
Copy link
Contributor

@slig3 slig3 commented Nov 27, 2018

Be able to select a step in a .feature file and rename it using Visual Studio rename shortcut.
The user is prompted to modify the regex in the binding attribute of the step definition.
Then, the modification on the regex is applied on all the steps that are bound to this method.
The values specified in the step names are conserved after the name change.

Catherine Tetreault-Beaulieu added 2 commits November 27, 2018 11:08
Modify error message when no or too many bindings have been found.
Replace only regex that perfectly match the rename, not the others.
@SabotageAndi
Copy link
Contributor

@slig3 Thanks for the PR. This would be a nice feature for the VS Extension.
Is this PR finished to look at?

@slig3
Copy link
Contributor Author

slig3 commented Dec 3, 2018

@SabotageAndi Yes, I finished to look at this PR. Should I close it or other people want to look at this?

@SabotageAndi
Copy link
Contributor

@slig3 I will have a look at it and will merge it or give you some comments.
Please don't close the PR, because then I can't merge it.

@SabotageAndi
Copy link
Contributor

Btw, it could take some days until I find the time to do the review

@slig3
Copy link
Contributor Author

slig3 commented Dec 3, 2018

No problem. Thanks a lot!

@slig3
Copy link
Contributor Author

slig3 commented Jan 16, 2019

@SabotageAndi Hi, I was wondering if you had time to look at this PR?

@SabotageAndi
Copy link
Contributor

facepalm

Oops. Sorry, I forgot this PR.
I will look at it next Monday.

@SabotageAndi SabotageAndi merged commit ceadcf7 into SpecFlowOSS:master Jan 21, 2019
@SabotageAndi
Copy link
Contributor

@slig3 So, I had a look at it.
It worked really good. Thanks for this PR. You will be in the hearth of a lot of SpecFlow users from now on.

@Stephen-Mc
Copy link

Thanks for your contribution to SpecFlow. If you would like us to send you some SpecFlow stickers as a thank you, please fill out the form here.

littlegenius666 pushed a commit to littlegenius666/SpecFlow.VisualStudio that referenced this pull request May 3, 2019
Be able to select a step in a .feature file and rename it using Visual Studio rename shortcut.
The user is prompted to modify the regex in the binding attribute of the step definition.
Then, the modification on the regex is applied on all the steps that are bound to this method.
The values specified in the step names are conserved after the name change.
@farazoman
Copy link

Before taking any next steps I wanted to check here to ask if the following is supported.

We use sentences with regex, so when I ran this renaming logic, it didn't end up preserving the text that would have matched the original regex. So is this expected? I wanted to look into it if not because its useful for our team

@SabotageAndi
Copy link
Contributor

@farazoman please create an issue at https://github.com/SpecFlowOSS/SpecFlow and add a concrete example for your steps there.
I am not 100% sure how your steps are. Thanks!

@farazoman
Copy link

farazoman commented Jul 13, 2020 via email

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