You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@maven.apache.org by GitBox <gi...@apache.org> on 2021/01/07 11:16:58 UTC

[GitHub] [maven-doxia] elharo opened a new pull request #53: rollback Java 7 breaking change

elharo opened a new pull request #53:
URL: https://github.com/apache/maven-doxia/pull/53


   @michael-o @bertysentry 


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



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org


[GitHub] [maven-doxia] bertysentry commented on pull request #53: rollback Java 7 breaking change

Posted by GitBox <gi...@apache.org>.
bertysentry commented on pull request #53:
URL: https://github.com/apache/maven-doxia/pull/53#issuecomment-756111986


   No problem. Remember we also have [this PR](https://github.com/apache/maven-doxia/pull/54) that restores the Java 7 compatibility. Thanks!


----------------------------------------------------------------
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] [maven-doxia] bertysentry commented on pull request #53: rollback Java 7 breaking change

Posted by GitBox <gi...@apache.org>.
bertysentry commented on pull request #53:
URL: https://github.com/apache/maven-doxia/pull/53#issuecomment-756059737


   Personally, I'm in favor of simply adding a commit that fixes the problem rather than reverting and then recommitting the same code plus the fix. But that's a git philosophy question, so I'll let you guys decide ;-)


----------------------------------------------------------------
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] [maven-doxia] elharo commented on pull request #53: rollback Java 7 breaking change

Posted by GitBox <gi...@apache.org>.
elharo commented on pull request #53:
URL: https://github.com/apache/maven-doxia/pull/53#issuecomment-756061601


   That's a moot point until such a fix exists. 


----------------------------------------------------------------
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] [maven-doxia] bertysentry commented on pull request #53: rollback Java 7 breaking change

Posted by GitBox <gi...@apache.org>.
bertysentry commented on pull request #53:
URL: https://github.com/apache/maven-doxia/pull/53#issuecomment-756062054


   I have it. I'll submit my PR in a moment.


----------------------------------------------------------------
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] [maven-doxia] elharo closed pull request #53: rollback Java 7 breaking change

Posted by GitBox <gi...@apache.org>.
elharo closed pull request #53:
URL: https://github.com/apache/maven-doxia/pull/53


   


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



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org


[GitHub] [maven-doxia] elharo closed pull request #53: rollback Java 7 breaking change

Posted by GitBox <gi...@apache.org>.
elharo closed pull request #53:
URL: https://github.com/apache/maven-doxia/pull/53


   


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



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org


[GitHub] [maven-doxia] michael-o commented on pull request #53: rollback Java 7 breaking change

Posted by GitBox <gi...@apache.org>.
michael-o commented on pull request #53:
URL: https://github.com/apache/maven-doxia/pull/53#issuecomment-756104056


   Yepp, I need also a day or two to digest.


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