grunt-text-replace
grunt-text-replace copied to clipboard
Adds options to control overwiting behaviors.
forceCopy, defaults to true. forceOverwrite, defaults to false.
Hi there. First I wanted to say thanks for taking the time to write this and submit your pull request. Sorry for the delay in getting back to you.
Before I accept it into the main branch I wanted to ask for the following:
- Can you explain the intended usecase? What do you see as the benefit of forcing the copy to true or false - isn't the final result the same?
- If the result isn't the same, then it will be necessary to update the test case, to make sure the we automatically test the functionality & don't loose it in the future.
Many thanks again & regards,
Jonathan
Hi again,
Sorry there was a mix-up with git issues, so I missed your original answer as for the use case, regarding the watch task. That's fine and a good point. Can you think of a good way to test this? I'd like to make sure all functionality is covered by tests.
Thanks,
Jonathan