You are viewing a plain text version of this content. The canonical link for it is here.
Posted to gitbox@yetus.apache.org by GitBox <gi...@apache.org> on 2020/10/07 04:31:48 UTC

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

aw-was-here opened a new pull request #149:
URL: https://github.com/apache/yetus/pull/149


   


----------------------------------------------------------------
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

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

Posted by GitBox <gi...@apache.org>.
aw-was-here commented on pull request #149:
URL: https://github.com/apache/yetus/pull/149#issuecomment-705329438






----------------------------------------------------------------
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

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

Posted by GitBox <gi...@apache.org>.
aw-zededa commented on pull request #149:
URL: https://github.com/apache/yetus/pull/149#issuecomment-705683148


   Just so it's officially noted here, I've updated the release note in JIRA.


----------------------------------------------------------------
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

[GitHub] [yetus] aw-zededa removed a comment on pull request #149: YETUS-1019. Replace whitespace with blanks

Posted by GitBox <gi...@apache.org>.
aw-zededa removed a comment on pull request #149:
URL: https://github.com/apache/yetus/pull/149#issuecomment-705683148


   Just so it's officially noted here, I've updated the release note in JIRA.


----------------------------------------------------------------
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

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

Posted by GitBox <gi...@apache.org>.
aw-zededa commented on pull request #149:
URL: https://github.com/apache/yetus/pull/149#issuecomment-705683148


   Just so it's officially noted here, I've updated the release note in JIRA.


----------------------------------------------------------------
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

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

Posted by GitBox <gi...@apache.org>.
aw-was-here commented on pull request #149:
URL: https://github.com/apache/yetus/pull/149#issuecomment-705841212


   Thanks! I'll merge this in now...
   


----------------------------------------------------------------
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

[GitHub] [yetus] aw-zededa removed a comment on pull request #149: YETUS-1019. Replace whitespace with blanks

Posted by GitBox <gi...@apache.org>.
aw-zededa removed a comment on pull request #149:
URL: https://github.com/apache/yetus/pull/149#issuecomment-705683148


   Just so it's officially noted here, I've updated the release note in JIRA.


----------------------------------------------------------------
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

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

Posted by 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

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

Posted by GitBox <gi...@apache.org>.
aw-was-here merged pull request #149:
URL: https://github.com/apache/yetus/pull/149


   


----------------------------------------------------------------
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

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

Posted by GitBox <gi...@apache.org>.
aw-was-here merged pull request #149:
URL: https://github.com/apache/yetus/pull/149


   


----------------------------------------------------------------
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

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

Posted by GitBox <gi...@apache.org>.
aw-was-here commented on pull request #149:
URL: https://github.com/apache/yetus/pull/149#issuecomment-705695876


   Just so it is noted here, I've updated the release notes in JIRA.


----------------------------------------------------------------
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