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 ASM to 9.7 to support class files generated for Java 23 #284

Merged
merged 1 commit into from
Dec 18, 2024

Conversation

fzhinkin
Copy link
Collaborator

No description provided.

@fzhinkin fzhinkin marked this pull request as ready for review December 18, 2024 16:07
@fzhinkin fzhinkin requested a review from shanshin December 18, 2024 16:08
@fzhinkin
Copy link
Collaborator Author

@shanshin and as you envisioned, compatibility tests are pretty useless when it comes to catching these errors.

@shanshin
Copy link
Contributor

@fzhinkin, do you know if there is a new version of the org.jetbrains.intellij.deps:asm-all artifact?

@fzhinkin
Copy link
Collaborator Author

Nope, 9.6.1 is the latest. But class file version check is disabled there, so it works with class files from JDK23+ despite being a bit outdated.

@fzhinkin fzhinkin merged commit da04a5e into develop Dec 18, 2024
3 checks passed
@fzhinkin fzhinkin deleted the asm-version-update branch December 18, 2024 18:49
@JakeWharton
Copy link

This also should support JDK 24, too!

svc-squareup-copybara pushed a commit to cashapp/misk that referenced this pull request Dec 22, 2024
| Package | Type | Package file | Manager | Update | Change |
|---|---|---|---|---|---|
|
[org.assertj:assertj-core](https://assertj.github.io/doc/#assertj-core)
([source](https://github.com/assertj/assertj)) | dependencies |
misk/gradle/libs.versions.toml | gradle | minor | `3.26.3` -> `3.27.0` |
|
[org.jetbrains.kotlinx.binary-compatibility-validator](https://github.com/Kotlin/binary-compatibility-validator)
| plugin | misk/gradle/libs.versions.toml | gradle | minor | `0.16.3` ->
`0.17.0` |
| [software.amazon.awssdk:sdk-core](https://aws.amazon.com/sdkforjava) |
dependencies | misk/gradle/libs.versions.toml | gradle | patch |
`2.29.37` -> `2.29.39` |
|
[software.amazon.awssdk:dynamodb-enhanced](https://aws.amazon.com/sdkforjava)
| dependencies | misk/gradle/libs.versions.toml | gradle | patch |
`2.29.37` -> `2.29.39` |
| [software.amazon.awssdk:dynamodb](https://aws.amazon.com/sdkforjava) |
dependencies | misk/gradle/libs.versions.toml | gradle | patch |
`2.29.37` -> `2.29.39` |
| [software.amazon.awssdk:aws-core](https://aws.amazon.com/sdkforjava) |
dependencies | misk/gradle/libs.versions.toml | gradle | patch |
`2.29.37` -> `2.29.39` |
| [software.amazon.awssdk:bom](https://aws.amazon.com/sdkforjava) |
dependencies | misk/gradle/libs.versions.toml | gradle | patch |
`2.29.37` -> `2.29.39` |
| [software.amazon.awssdk:auth](https://aws.amazon.com/sdkforjava) |
dependencies | misk/gradle/libs.versions.toml | gradle | patch |
`2.29.37` -> `2.29.39` |

---

### Release Notes

<details>
<summary>Kotlin/binary-compatibility-validator
(org.jetbrains.kotlinx.binary-compatibility-validator)</summary>

###
[`v0.17.0`](https://github.com/Kotlin/binary-compatibility-validator/releases/tag/0.17.0)

[Compare
Source](Kotlin/binary-compatibility-validator@0.16.3...0.17.0)

#### What's Changed

- `$suspendImpl` functions are now filtered from dumps
([#&#8203;271](Kotlin/binary-compatibility-validator#271))
This change may cause validation failures for projects where dumps
already contain such functions. It is recommended to regenerate dumps
(`apiDump` task) after updating the BCV to `0.17.0`.
- Supported class files generated for JVM target 23
([#&#8203;284](Kotlin/binary-compatibility-validator#284))

</details>

---

### Configuration

📅 **Schedule**: Branch creation - "after 6pm every weekday,before 2am
every weekday" in timezone Australia/Melbourne, Automerge - At any time
(no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you
are satisfied.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

👻 **Immortal**: This PR will be recreated if closed unmerged. Get
[config help](https://github.com/renovatebot/renovate/discussions) if
that's undesired.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR has been generated by [Renovate
Bot](https://github.com/renovatebot/renovate).

GitOrigin-RevId: 93958c08cfec442c4c0ece414e7da1959e0e9aa9
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