Skip to content

Update Kotlin to 1.8.22, Gradle to 8.2.1, update related tooling#1974

Merged
dd-mergequeue[bot] merged 1 commit into
developfrom
nogorodnikov/update-kotlin-to-1.8.22
Apr 9, 2024
Merged

Update Kotlin to 1.8.22, Gradle to 8.2.1, update related tooling#1974
dd-mergequeue[bot] merged 1 commit into
developfrom
nogorodnikov/update-kotlin-to-1.8.22

Conversation

@0xnm

@0xnm 0xnm commented Apr 8, 2024

Copy link
Copy Markdown
Member

What does this PR do?

This PR updates Kotlin to 1.8.22, Gradle to 8.2.1 and also updates related tooling.

Latest versions of Kotlin (1.9) and Gradle (8.7) are not used, because Kotlin 1.9 may bring backward binary incompatibility (to be checked), so this PR is on the safe side and uses versions of the build tooling (Gradle included) which are still on Kotlin 1.8 or below.

Motivation

What inspired you to submit this pull request?

Additional Notes

Anything else we should know when reviewing?

Review checklist (to be filled by reviewers)

  • Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
  • Make sure you discussed the feature or bugfix with the maintaining team in an Issue
  • Make sure each commit and the PR mention the Issue number (cf the CONTRIBUTING doc)

@0xnm
0xnm force-pushed the nogorodnikov/update-kotlin-to-1.8.22 branch 3 times, most recently from 10b6914 to f71ea43 Compare April 8, 2024 13:25
@codecov-commenter

codecov-commenter commented Apr 8, 2024

Copy link
Copy Markdown

Codecov Report

Merging #1974 (166a522) into develop (ccc9e1f) will increase coverage by 0.23%.
Report is 10 commits behind head on develop.
The diff coverage is n/a.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #1974      +/-   ##
===========================================
+ Coverage    83.38%   83.61%   +0.23%     
===========================================
  Files          489      487       -2     
  Lines        17925    17742     -183     
  Branches      2670     2664       -6     
===========================================
- Hits         14945    14834     -111     
+ Misses        2249     2187      -62     
+ Partials       731      721      -10     
Files Coverage Δ
...ssionreplay/internal/recorder/ViewUtilsInternal.kt 100.00% <ø> (ø)

... and 119 files with indirect coverage changes

@0xnm
0xnm force-pushed the nogorodnikov/update-kotlin-to-1.8.22 branch from f71ea43 to 166a522 Compare April 8, 2024 13:59
@0xnm
0xnm marked this pull request as ready for review April 8, 2024 14:01
@0xnm
0xnm requested review from a team as code owners April 8, 2024 14:01
Base automatically changed from nogorodnikov/rum-3486/remove-non-ascii-characters-from-test-names to develop April 9, 2024 07:56
@0xnm

0xnm commented Apr 9, 2024

Copy link
Copy Markdown
Member Author

/merge

@dd-devflow

dd-devflow Bot commented Apr 9, 2024

Copy link
Copy Markdown

🚂 MergeQueue

This merge request is not mergeable yet, because of pending checks/missing approvals. It will be added to the queue as soon as checks pass and/or get approvals.
Note: if you pushed new commits since the last approval, you may need additional approval.
You can remove it from the waiting list with /remove command.

Use /merge -c to cancel this operation!

@dd-devflow

dd-devflow Bot commented Apr 9, 2024

Copy link
Copy Markdown

🚂 MergeQueue

Pull request added to the queue.

This build is going to start soon! (estimated merge in less than 0s)

Use /merge -c to cancel this operation!

@dd-mergequeue
dd-mergequeue Bot merged commit 2896bdb into develop Apr 9, 2024
@dd-mergequeue
dd-mergequeue Bot deleted the nogorodnikov/update-kotlin-to-1.8.22 branch April 9, 2024 08:48
@xgouchet xgouchet added this to the 2.9.X milestone Jul 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants