Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Added mvn check #569

Merged
merged 1 commit into from
Jun 11, 2024
Merged

Added mvn check #569

merged 1 commit into from
Jun 11, 2024

Conversation

sophiaxu0424
Copy link

Issue related to: #567

Added mvn check
ee803fe 
Issue related to: adoptium#567

Signed-off-by: Minghong Xu <Minghong.Xu@ibm.com>
@sophiaxu0424
Copy link
Author

Passed tests with maven installed:https://hyc-runtimes-jenkins.swg-devops.com/job/Grinder/41309/

14:51:35  mavenVersion : 
14:51:35  	Apache Maven 3.6.3 (cecedd343002696d0abb50b32b541b8a6ba2883f)
14:51:35  	Maven home: /usr/local/apache-maven-3.6.3

@sophiaxu0424
Copy link
Author

and mvn not installed on an aix platform here, but test passed: https://hyc-runtimes-jenkins.swg-devops.com/job/Grinder/41310/:

 mavenVersion : bash: line 1: mvn: command not found

@sophiaxu0424
Copy link
Author

@llxia @LongyuZhang please kindly review. thanks!

@sophiaxu0424 sophiaxu0424 marked this pull request as ready for review June 11, 2024 19:44
LongyuZhang
LongyuZhang approved these changes Jun 11, 2024
View reviewed changes
Copy link
Contributor

@LongyuZhang LongyuZhang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@LongyuZhang LongyuZhang merged commit 0a68df7 into adoptium:master Jun 11, 2024
3 checks passed
@sophiaxu0424 sophiaxu0424 deleted the mvn branch June 13, 2024 13:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@llxia llxia llxia approved these changes

@LongyuZhang LongyuZhang LongyuZhang approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@sophiaxu0424 @llxia @LongyuZhang