Skip to content
This repository was archived by the owner on Feb 9, 2022. It is now read-only.

Conversation

@rosen-vladimirov
Copy link
Contributor

Allow verification of the environment based on a specified Android runtime version. This is required for cases where the package.json of the project is not updated yet, but we need to verify that we can use this runtime

Allow verification of the environment based on a specified Android runtime version. This is required for cases where the package.json of the project is not updated yet, but we need to verify that we can use this runtime
@rosen-vladimirov rosen-vladimirov self-assigned this May 28, 2018
/**
* Checks if a local build can be executed on the current machine.
* @param {string} platform The platform for which to check if local build is possible.
* @param {string} runtimeVersion @optional The runtime version against which the validation is executed. In case this parameter is passed, it takes predescences over the projectDir argument.

Choose a reason for hiding this comment

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

There seems to be no projectDir argument here?

* @param {string} installedJavaVersion The version of javac to check.
* @param {string} projectDir @optional The project directory. Used to determine the Android Runtime version and validate the Java compiler version against it.
* If it is not passed or the project does not have Android runtime, this validation is skipped.
* @param {string} runtimeVersion @optional The runtime version against which the validation is executed. In case this parameter is passed, it takes predescences over the projectDir argument.

Choose a reason for hiding this comment

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

predescences -> precedence

Copy link

@Mitko-Kerezov Mitko-Kerezov left a comment

Choose a reason for hiding this comment

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

Looks all right to me

@rosen-vladimirov rosen-vladimirov merged commit 7cc3b06 into master May 28, 2018
@rosen-vladimirov rosen-vladimirov deleted the vladimirov/validate-against-runtime branch May 28, 2018 13:46
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants