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

Re-enable JTReg test thrinfo01 #4174

Merged
merged 1 commit into from
Dec 1, 2022

Conversation

dipak-bagadiya
Copy link
Contributor

@dipak-bagadiya dipak-bagadiya commented Nov 30, 2022

Re-enable JTReg test thrinfo01

Test had been fixed by eclipse-openj9/openj9#16376

Signed-off-by: Dipak Bagadiya dipak.bagadiya@ibm.com

@llxia
Copy link
Contributor

llxia commented Nov 30, 2022

Related: eclipse-openj9/openj9#16226

Copy link
Contributor Author

@dipak-bagadiya dipak-bagadiya left a comment

Choose a reason for hiding this comment

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

@fengxue-IS @babsingh - please review

@fengxue-IS
Copy link
Contributor

fengxue-IS commented Nov 30, 2022

please update the commit message to something like

Re-enable JTReg test thrinfo01

Test has been fix by https://github.com/eclipse-openj9/openj9/pull/16376

Signed-off-by: <your email>

Please also update the title of this PR. Convension is that the reference to openj9 PR should be part of the message, not title.

@dipak-bagadiya dipak-bagadiya changed the title Test has been fix by-https://github.com/eclipse-openj9/openj9/pull/16376 Re-enable JTReg test thrinfo01 Dec 1, 2022
@dipak-bagadiya
Copy link
Contributor Author

@fengxue-IS -> I updated the title and comment as needed.

Copy link
Contributor

@Mesbah-Alam Mesbah-Alam 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
Copy link
Contributor

llxia commented Dec 1, 2022

@dipak-bagadiya could you also update the commit message? Thanks

@babsingh
Copy link
Contributor

babsingh commented Dec 1, 2022

Also, it should be past tense: Test has been fixed by eclipse-openj9/openj9#16376.

Test had been fixed by-
eclipse-openj9/openj9#16376

Signed-off-by: Dipak Bagadiya dipak.bagadiya@ibm.com
@dipak-bagadiya
Copy link
Contributor Author

@fengxue-IS @babsingh @llxia I updated the commit message and made the past tense in the description part.

@llxia llxia merged commit 4789c7d into adoptium:master Dec 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants