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

Update Gradle version 8.7 and java version 17 within onnxruntime/java #22771

Merged
merged 14 commits into from
Nov 15, 2024

Conversation

jchen351
Copy link
Contributor

@jchen351 jchen351 commented Nov 8, 2024

Description

This change is to update the Gradle version within java project to 8.7, it also upgrades the JAVA to 17. Gradle version from react-native was also updated to 7.5 to make it compatible with changes from the Java directory. However, the target java version remains the same. Java version from these will be upgraded in a separated PR.

This is spited from #22206

Motivation and Context

This is the first step to upgrade the react native version.

@jchen351 jchen351 requested a review from a team as a code owner November 14, 2024 18:43
Copy link
Member

@snnn snnn left a comment

Choose a reason for hiding this comment

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

Thanks!

@jchen351 jchen351 merged commit 632a36a into main Nov 15, 2024
93 checks passed
@jchen351 jchen351 deleted the Cjian/java_gradle branch November 15, 2024 01:10
ishwar-raut1 pushed a commit to ishwar-raut1/onnxruntime that referenced this pull request Nov 19, 2024
…microsoft#22771)

### Description
This change is to update the Gradle version within java project to 8.7,
it also upgrades the JAVA to 17. Gradle version from react-native was
also updated to 7.5 to make it compatible with changes from the Java
directory. However, the target java version remains the same. Java
version from these will be upgraded in a separated PR.

This is spited from microsoft#22206

### Motivation and Context
This is the first step to upgrade the react native version.
mszhanyi pushed a commit that referenced this pull request Nov 21, 2024
mszhanyi pushed a commit that referenced this pull request Nov 21, 2024
mszhanyi added a commit that referenced this pull request Nov 21, 2024
#22914)

…ime/java (#22771)"

This reverts commit 632a36a.

### Description
<!-- Describe your changes. -->



### Motivation and Context
Run E2E tests using Browserstack failed due to this PR.
@jchen351 jchen351 restored the Cjian/java_gradle branch November 21, 2024 23:19
mszhanyi added a commit that referenced this pull request Nov 22, 2024
#22914)

…ime/java (#22771)"

This reverts commit 632a36a.

### Description
<!-- Describe your changes. -->



### Motivation and Context
Run E2E tests using Browserstack failed due to this PR.
guschmue pushed a commit that referenced this pull request Dec 2, 2024
…#22771)

### Description
This change is to update the Gradle version within java project to 8.7,
it also upgrades the JAVA to 17. Gradle version from react-native was
also updated to 7.5 to make it compatible with changes from the Java
directory. However, the target java version remains the same. Java
version from these will be upgraded in a separated PR.

This is spited from #22206

### Motivation and Context
This is the first step to upgrade the react native version.
guschmue pushed a commit that referenced this pull request Dec 2, 2024
#22914)

…ime/java (#22771)"

This reverts commit 632a36a.

### Description
<!-- Describe your changes. -->



### Motivation and Context
Run E2E tests using Browserstack failed due to this PR.
ankitm3k pushed a commit to intel/onnxruntime that referenced this pull request Dec 11, 2024
…microsoft#22771)

### Description
This change is to update the Gradle version within java project to 8.7,
it also upgrades the JAVA to 17. Gradle version from react-native was
also updated to 7.5 to make it compatible with changes from the Java
directory. However, the target java version remains the same. Java
version from these will be upgraded in a separated PR.

This is spited from microsoft#22206

### Motivation and Context
This is the first step to upgrade the react native version.
ankitm3k pushed a commit to intel/onnxruntime that referenced this pull request Dec 11, 2024
microsoft#22914)

…ime/java (microsoft#22771)"

This reverts commit 632a36a.

### Description
<!-- Describe your changes. -->



### Motivation and Context
Run E2E tests using Browserstack failed due to this PR.
ankitm3k pushed a commit to intel/onnxruntime that referenced this pull request Dec 11, 2024
…microsoft#22771)

### Description
This change is to update the Gradle version within java project to 8.7,
it also upgrades the JAVA to 17. Gradle version from react-native was
also updated to 7.5 to make it compatible with changes from the Java
directory. However, the target java version remains the same. Java
version from these will be upgraded in a separated PR.

This is spited from microsoft#22206

### Motivation and Context
This is the first step to upgrade the react native version.
ankitm3k pushed a commit to intel/onnxruntime that referenced this pull request Dec 11, 2024
microsoft#22914)

…ime/java (microsoft#22771)"

This reverts commit 632a36a.

### Description
<!-- Describe your changes. -->



### Motivation and Context
Run E2E tests using Browserstack failed due to this PR.
ankitm3k pushed a commit to intel/onnxruntime that referenced this pull request Dec 11, 2024
…microsoft#22771)

### Description
This change is to update the Gradle version within java project to 8.7,
it also upgrades the JAVA to 17. Gradle version from react-native was
also updated to 7.5 to make it compatible with changes from the Java
directory. However, the target java version remains the same. Java
version from these will be upgraded in a separated PR.

This is spited from microsoft#22206

### Motivation and Context
This is the first step to upgrade the react native version.
ankitm3k pushed a commit to intel/onnxruntime that referenced this pull request Dec 11, 2024
microsoft#22914)

…ime/java (microsoft#22771)"

This reverts commit 632a36a.

### Description
<!-- Describe your changes. -->



### Motivation and Context
Run E2E tests using Browserstack failed due to this PR.
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.

3 participants