You are viewing a plain text version of this content. The canonical link for it is here.

Posted to notifications@skywalking.apache.org by "kezhenxu94 (via GitHub)" <gi...@apache.org> on 2023/01/31 05:53:34 UTC

[GitHub] [skywalking-java] kezhenxu94 commented on pull request #444: Add plugin for Spring 6, set development baseline to JDK 17, remove Powermock entirely

kezhenxu94 commented on PR #444:
URL: https://github.com/apache/skywalking-java/pull/444#issuecomment-1409801597

   For all @apache/skywalking-committers , I've removed Powermock completely from skywalking-java repository and there is no reason we would bring it back in the future, I'll also try to remove it in the skywalking main repo. This is a future-prove work because Powermock would block us from upgrading the JDK baseline to Java8+, or upgrading the Junit framework to Junit 5. So be careful when you review contributors' pull requests, if they bring Powermock back, don't allow to merge.


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

To unsubscribe, e-mail: notifications-unsubscribe@skywalking.apache.org

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