Skip to content

Commit

Permalink
chore (doctor): update Java version range to be strictly 11-18
Browse files Browse the repository at this point in the history
  • Loading branch information
tarunrajput committed May 17, 2023
1 parent 29c7ea0 commit 6975395
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 5 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,7 @@ describe('jdk', () => {
expect(diagnostics.needsToBeFixed).toBe(true);
});

it('returns false if JDK version is in range (JDK 9+ version number format)', async () => {
it('returns false if JDK version is in range (JDK [11-18] version number format)', async () => {
// @ts-ignore
environmentInfo.Languages.Java = {
version: '14.0.4',
Expand All @@ -45,7 +45,7 @@ describe('jdk', () => {
expect(diagnostics.needsToBeFixed).toBe(false);
});

it('returns true if JDK version is not in range (JDK <= 8 version number format)', async () => {
it('returns true if JDK version is not in range (JDK < 11 version number format)', async () => {
// @ts-ignore
environmentInfo.Languages.Java = {
version: '1.8.0_282',
Expand All @@ -55,10 +55,10 @@ describe('jdk', () => {
expect(diagnostics.needsToBeFixed).toBe(true);
});

it('returns true if JDK version is not in range (JDK 9+ verison number format)', async () => {
it('returns true if JDK version is not in range (JDK > 18 version number format)', async () => {
// @ts-ignore
environmentInfo.Languages.Java = {
version: '10.0.15+10',
version: '19.0.1',
};

const diagnostics = await jdk.getDiagnostics(environmentInfo);
Expand Down
2 changes: 1 addition & 1 deletion packages/cli-doctor/src/tools/versionRanges.ts
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ export default {
YARN: '>= 1.10.x',
NPM: '>= 4.x',
RUBY: '>= 2.6.10',
JAVA: '>= 11',
JAVA: '>= 11 < 19',
// Android
ANDROID_SDK: '>= 33.x',
ANDROID_NDK: '>= 23.x',
Expand Down

0 comments on commit 6975395

Please sign in to comment.