kezhenxu94 commented on a change in pull request #6403:
URL: https://github.com/apache/skywalking/pull/6403#discussion_r579673429
##########
File path: test/e2e/pom.xml
##########
@@ -44,6 +44,8 @@
</modules>
<properties>
+ <sw.version></sw.version> <!-- Please manually set the SkyWalking
version here to build in the IDE, but make sure not to check it into the code
base / git -->
Review comment:
> Or VM option of the IDE, right?
>
> I still feel, we should add a check step in the CI, before we run maven.
Check what? The property is already empty.
----------------------------------------------------------------
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]
