Skip to content

Kotlin: Run the tests with 2.2.0 #20031

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 3 commits into from
Jul 22, 2025
Merged

Conversation

igfoo
Copy link
Contributor

@igfoo igfoo commented Jul 11, 2025

No description provided.

@igfoo igfoo force-pushed the igfoo/kotlin-tests-2.2.0 branch from 906019b to 4c4f7d6 Compare July 22, 2025 16:24
@igfoo igfoo force-pushed the igfoo/kotlin-tests-2.2.0 branch from 4c4f7d6 to cd3143f Compare July 22, 2025 16:38
@igfoo igfoo marked this pull request as ready for review July 22, 2025 17:50
@Copilot Copilot AI review requested due to automatic review settings July 22, 2025 17:50
@igfoo igfoo requested review from a team as code owners July 22, 2025 17:50
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This pull request updates the Kotlin tests to run with version 2.2.0, upgrading from the previous version 2.1.20. This appears to be part of ensuring compatibility and testing with the latest Kotlin compiler version.

  • Updated default Kotlin version from 2.1.20 to 2.2.0 in the extractor wrapper
  • Updated test expectations to reflect changes in compiler-generated code locations for enum classes
  • Temporarily disabled a custom plugin integration test pending updates for Kotlin 2.2 compatibility

Reviewed Changes

Copilot reviewed 10 out of 10 changed files in this pull request and generated no comments.

File Description
java/kotlin-extractor/dev/wrapper.py Updates default Kotlin version to 2.2.0
Multiple .expected files Updates test expectations for enum class compiler-generated methods with new location mappings
java/ql/integration-tests/kotlin/linux/custom_plugin/test.py-disabled Adds comment explaining temporary test disable for Kotlin 2.2 compatibility

Copy link
Contributor

@IdrissRio IdrissRio left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@igfoo igfoo merged commit 09dd708 into github:main Jul 22, 2025
15 checks passed
@igfoo igfoo deleted the igfoo/kotlin-tests-2.2.0 branch July 22, 2025 21:20
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.

2 participants
pFad - Phonifier reborn

Pfad - The Proxy pFad of © 2024 Garber Painting. All rights reserved.

Note: This service is not intended for secure transactions such as banking, social media, email, or purchasing. Use at your own risk. We assume no liability whatsoever for broken pages.


Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy