[GitHub] [yetus] aw-was-here commented on pull request #149: YETUS-1019. Replace whitespace with blanks

[GitBox <gi...@apache.org>]

aw-was-here commented on pull request #149:
URL: https://github.com/apache/yetus/pull/149#issuecomment-705329438


   > we should call out in the release note that the old whitespace cli option are deprecated and will get removed (unless we want to start having some kind of "you used a deprecated feature run with `--what-was-deprecated` for details" warning.)
   
   Good point. I'll add that to the release notes.
   
   > also still flag it as incompatible because if folks were enabling or disabling the plugin via its name as whitespace that won't work now? or do we still want that to work?
   
   Yup, I'll also add that to the release notes.  But yeah, as written the bw compat will only work for people who don't mention it by name (which AFAIK is the vast majority of people).


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org