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

Add GradleV3 task to fix invalid demands issue #15174

Merged
merged 3 commits into from
Aug 23, 2021

Conversation

DaniilShmelev
Copy link
Contributor

@DaniilShmelev DaniilShmelev commented Aug 19, 2021

Task name: Gradle

Description: Remove gradle demand; fix tests; add GradleV3 to make-options.json
Difference: DaniilShmelev/diff@03fb23a
Previously jobs with the Gradle task would not run even if JDK was installed on the agent. The only thing that's changed in V3 is that the java demand is removed.

Documentation changes required: Yes

Added unit tests: No (only fixed the old ones since they had hardcoded "GradleV2" strings)

Checklist:

  • Task version was bumped - please check instruction how to do it
  • Checked that applied changes work as expected

@DaniilShmelev DaniilShmelev self-assigned this Aug 19, 2021
@DaniilShmelev DaniilShmelev marked this pull request as ready for review August 19, 2021 08:14
@DaniilShmelev DaniilShmelev requested a review from a team August 19, 2021 08:15
@anatolybolshakov
Copy link
Contributor

LGTM, thanks! Could it make sense to update related localization files also in Localization branch?

@DaniilShmelev
Copy link
Contributor Author

@anatolybolshakov Sure, I've created a PR: #15178

@anatolybolshakov anatolybolshakov requested a review from a team August 20, 2021 16:03
@kuleshovilya
Copy link
Contributor

Tested functionality manually, LGTM!

@DaniilShmelev DaniilShmelev merged commit 9363dd7 into master Aug 23, 2021
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