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:
[email protected]


Reply via email to