Skip to content

Updated gradle configuration #764

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 4 commits into from
Aug 26, 2022
Merged

Conversation

Markoutte
Copy link
Collaborator

@Markoutte Markoutte commented Aug 23, 2022

Description

Several changes are done to gradle project model:

  1. Set Java 8 by default to all modules
  2. Set Java 11 for utbot-intellij module but Java 8 for source. Motivation: To keep all code-base in Java 8 but make it usable by newer version of Intellij
  3. Removed all dependencies on task .tasks.jar. Actually, I don't know exactly, why we need them
  4. Removed -Werror for java compilation task

This PR fixes an issue with red code for fuzzer module from framework and analytics module.

How Has This Been Tested?

Automated Testing

All existing tests are passed both: locally and with actions.

Manual Scenario

Tried project with 8 and 11 versions of Java. Both generated correct tests.

@Markoutte Markoutte force-pushed the pelevin/gradle-version-changes branch from f473985 to 69b5ac1 Compare August 24, 2022 08:22
@Markoutte Markoutte requested review from denis-fokin and dtim August 24, 2022 11:50
@Markoutte Markoutte marked this pull request as ready for review August 24, 2022 11:51
@Markoutte Markoutte requested a review from amandelpie August 25, 2022 07:08
@Markoutte Markoutte force-pushed the pelevin/gradle-version-changes branch from 1c8770c to f75fd48 Compare August 26, 2022 07:25
@Markoutte Markoutte requested review from amandelpie and removed request for amandelpie August 26, 2022 08:12
@denis-fokin denis-fokin removed the request for review from amandelpie August 26, 2022 08:21
@denis-fokin denis-fokin dismissed amandelpie’s stale review August 26, 2022 08:23

All changes are resolved

@Markoutte Markoutte merged commit 73b6e89 into main Aug 26, 2022
@Markoutte Markoutte deleted the pelevin/gradle-version-changes branch August 26, 2022 08:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants