Skip to content

fix: During all gradle operations "Gradle build" message is printed #3744

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

Merged
merged 1 commit into from
Jul 12, 2018

Conversation

rosen-vladimirov
Copy link
Contributor

In case CLI executes any Gradle operation, it always prints Gradle build... message. Fix this by printing specific message for each operation, for example "Gradle clean..." when clean is executed.
Also make some methods private as they are used only in AndroidProjectService and they were incorrectly added to the IPlatformProjectService interface.

PR Checklist

What is the current behavior?

When CLI calls gradle clean (for example if you call tns build android --bundle && tns build android), Gradle build... is printed, while executing clean operation.

What is the new behavior?

CLI prints correct information for the gradle task.

In case CLI executes any Gradle operation, it always prints `Gradle build...` message. Fix this by printing specific message for each operation, for example "Gradle clean..." when clean is executed.
Also make some methods private as they are used only in AndroidProjectService and they were incorrectly added to the IPlatformProjectService interface.
@NativeScript NativeScript deleted a comment from DimitarTachev Jul 12, 2018
@NativeScript NativeScript deleted a comment from rosen-vladimirov Jul 12, 2018
dtopuzov added a commit to NativeScript/nativescript-cli-tests that referenced this pull request Jul 12, 2018
@dtopuzov
Copy link
Contributor

run ci

@rosen-vladimirov rosen-vladimirov merged commit e83918b into master Jul 12, 2018
@rosen-vladimirov rosen-vladimirov deleted the vladimirov/fix-gradle-messages branch July 12, 2018 14:15
dtopuzov added a commit to NativeScript/nativescript-cli-tests that referenced this pull request Jul 12, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants