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

Update Gradle version to 8 #3662

Merged
merged 1 commit into from
Jul 13, 2023
Merged

Update Gradle version to 8 #3662

merged 1 commit into from
Jul 13, 2023

Conversation

e5l
Copy link
Member

@e5l e5l commented Jun 27, 2023

No description provided.

@e5l e5l self-assigned this Jun 27, 2023
@e5l e5l marked this pull request as ready for review June 28, 2023 08:38
@e5l e5l requested a review from rsinukov June 28, 2023 08:38
Copy link
Contributor

@rsinukov rsinukov left a comment

Choose a reason for hiding this comment

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

Please check CI

buildSrc/build.gradle.kts Outdated Show resolved Hide resolved
@rsinukov
Copy link
Contributor

rsinukov commented Jul 7, 2023

Please rebase

@e5l
Copy link
Member Author

e5l commented Jul 7, 2023

done

Copy link
Contributor

@rsinukov rsinukov left a comment

Choose a reason for hiding this comment

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

LGTM, but please wait for the CI to run

@e5l e5l merged commit 6db8d13 into 3.0.0-eap Jul 13, 2023
@e5l e5l deleted the e5l/gradle8 branch July 13, 2023 06:58
e5l added a commit that referenced this pull request Jul 19, 2023
e5l added a commit that referenced this pull request Jul 26, 2023
e5l added a commit that referenced this pull request Jul 31, 2023
e5l added a commit that referenced this pull request Aug 7, 2023
e5l added a commit that referenced this pull request Aug 8, 2023
id("org.gradle.kotlin.kotlin-dsl") version "3.2.4"
kotlin("plugin.serialization") version "1.7.21"
kotlin("plugin.serialization") version "1.8.20"
id("org.gradle.kotlin.kotlin-dsl") version "4.0.14"
Copy link
Contributor

Choose a reason for hiding this comment

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

you should not set this version explicitly, but use kotlin-dsl. the same for plugin-serialization should use embeddedKotlinVersion.

@hfhbd
Copy link
Contributor

hfhbd commented Aug 8, 2023

Did you also run the task twice? The wrapper wasn't update too. Wanna have a PR?

@e5l
Copy link
Member Author

e5l commented Aug 8, 2023

Strange, will take a look

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants