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

Fix "substract" typo #6840

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Spencer-Comin
Copy link
Contributor

@Spencer-Comin Spencer-Comin commented Dec 7, 2022

substract -> subtract

@r30shah
Copy link
Contributor

r30shah commented Dec 9, 2022

Jenkins Build zos,zlinux

Copy link
Contributor

@r30shah r30shah left a comment

Choose a reason for hiding this comment

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

LGTM

@r30shah
Copy link
Contributor

r30shah commented May 3, 2023

Jenkins Build zos,zlinux

@r30shah
Copy link
Contributor

r30shah commented May 4, 2023

All the builds have passed. This one is good to merge. @0xdaryl Can I request to merge this one ?

@Spencer-Comin
Copy link
Contributor Author

Rebased due to eclipse-openj9/openj9#16431 (comment)

@r30shah
Copy link
Contributor

r30shah commented Nov 17, 2023

@Spencer-Comin Looks like this one was missed. Can you rebase your branch so we can start tests again and merge ?

@Spencer-Comin Spencer-Comin force-pushed the fix-substract-typo branch 2 times, most recently from 33796c9 to c50c8ae Compare November 17, 2023 20:34
@r30shah
Copy link
Contributor

r30shah commented Mar 12, 2024

@Spencer-Comin Do we need to rebase and get this merged?

Signed-off-by: Spencer Comin <spencer.comin@ibm.com>
@Spencer-Comin
Copy link
Contributor Author

Rebased and added remaining instances of "substract" throughout the codebase

@knn-k
Copy link
Contributor

knn-k commented Mar 13, 2024

The arm part looks good to me.

@r30shah
Copy link
Contributor

r30shah commented Mar 14, 2024

jenkins build all

@r30shah
Copy link
Contributor

r30shah commented Mar 14, 2024

@Spencer-Comin What is the order of this and the OpenJ9 PR to go ? Do we need coordinated merge ?

@Spencer-Comin
Copy link
Contributor Author

@r30shah we will need coordinated merge

@Spencer-Comin
Copy link
Contributor Author

x86 macOS failure looks like #7181

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