grunt-text-replace icon indicating copy to clipboard operation
grunt-text-replace copied to clipboard

Adds options to control overwiting behaviors.

Open old9 opened this issue 12 years ago • 2 comments

forceCopy, defaults to true. forceOverwrite, defaults to false.

old9 avatar Nov 08 '13 07:11 old9

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

yoniholmes avatar Feb 09 '14 10:02 yoniholmes

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

yoniholmes avatar Feb 09 '14 10:02 yoniholmes