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.

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

Enable runaqa PR build #12870

Merged
merged 1 commit into from
Jun 10, 2021
Merged

Enable runaqa PR build #12870

merged 1 commit into from
Jun 10, 2021

Conversation

renfeiw
Copy link
Contributor

@renfeiw renfeiw commented Jun 6, 2021

  • tailor runaqa action and workflow files for openj9
  • current runaqa version doesn't support build openj9

[ci skip]

Signed-off-by: renfeiw renfeiw@ca.ibm.com

@renfeiw
Copy link
Contributor Author

renfeiw commented Jun 6, 2021

This PR will work util the updated run-aqa action is released. The update is in adoptium/run-aqa#87

@renfeiw
Copy link
Contributor Author

renfeiw commented Jun 6, 2021

reviewers: @llxia @smlambert

- tailor runaqa action and workflow files for openj9
- current runaqa version doesn't support build openj9

[ci skip]

Signed-off-by: renfeiw <renfeiw@ca.ibm.com>
@renfeiw
Copy link
Contributor Author

renfeiw commented Jun 6, 2021

Copyright check is failed due to the configuration json file, which I assume that we don't need.

@renfeiw renfeiw marked this pull request as ready for review June 9, 2021 21:41
@renfeiw
Copy link
Contributor Author

renfeiw commented Jun 9, 2021

The new run-aqa action is realeased, this PR is ready to merge.

@llxia llxia added the comp:test label Jun 9, 2021
Copy link
Contributor

@llxia llxia left a comment

Choose a reason for hiding this comment

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

LGTM

@llxia llxia requested a review from smlambert June 9, 2021 21:43
Copy link
Contributor

@smlambert smlambert left a comment

Choose a reason for hiding this comment

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

Yay!

@smlambert smlambert merged commit 5d74198 into eclipse-openj9:master Jun 10, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants