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.

Sign up for GitHub

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

Jump to bottom

[build] update to kotlin 1.7.10 #1572

Merged
merged 10 commits into from
Oct 12, 2022
Merged

[build] update to kotlin 1.7.10 #1572

merged 10 commits into from
Oct 12, 2022
+97 −135

Conversation

samuelAndalon
Copy link
Contributor

@samuelAndalon samuelAndalon commented Oct 7, 2022 •
edited
Loading

@samuelAndalon samuelAndalon changed the title Migrate to kotlin 1.7 dependencies: Migrate to kotlin 1.7 Oct 7, 2022
@samuelAndalon samuelAndalon changed the title dependencies: Migrate to kotlin 1.7 dependencies: kotlin 1.7.10 Oct 8, 2022
@dariuszkuc
remove explicit jvm target from the plugin tests
953df16 
Unsure what is causing the classpath issue but this was the easiest fix.
@samuelAndalon samuelAndalon marked this pull request as ready for review October 12, 2022 05:00
@samuelAndalon samuelAndalon added type: documentation Documentation or test changes type: dependency changes Dependency change which doesn't affect our library usage labels Oct 12, 2022
@samuelAndalon samuelAndalon changed the title dependencies: kotlin 1.7.10 [build] update to kotlin 1.7.10 Oct 12, 2022
@samuelAndalon samuelAndalon mentioned this pull request Oct 12, 2022
Closed
dariuszkuc
dariuszkuc approved these changes Oct 12, 2022
View reviewed changes
Copy link
Collaborator

@dariuszkuc dariuszkuc left a comment

Choose a reason for hiding this comment

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

Looks like there were some unrelated changes (in instrumentation tests and context extension) but otherwise looks fine.

@samuelAndalon samuelAndalon merged commit c38d88b into master Oct 12, 2022
@samuelAndalon samuelAndalon deleted the migrate-to-kotlin-1.7 branch October 12, 2022 17:51
@dariuszkuc dariuszkuc mentioned this pull request Oct 19, 2022
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dariuszkuc dariuszkuc

@tapaderster tapaderster

@Samjin Samjin

Assignees
No one assigned
Labels
type: dependency changes Dependency change which doesn't affect our library usage type: documentation Documentation or test changes
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@samuelAndalon @dariuszkuc @jdvr