Skip to content

Fix false positive for const val with non-public marker#245

Merged
fzhinkin merged 4 commits intoKotlin:developfrom
martinbonnin:const-false-positive
Jun 28, 2024
Merged

Fix false positive for const val with non-public marker#245
fzhinkin merged 4 commits intoKotlin:developfrom
martinbonnin:const-false-positive

Conversation

@martinbonnin
Copy link
Copy Markdown
Contributor

Attempt at fixing #90.

Copy link
Copy Markdown
Collaborator

@fzhinkin fzhinkin left a comment

Choose a reason for hiding this comment

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

@martinbonnin, thanks for addressing the issue! Overall, the change looks good, I have just a few minor nits.

Comment thread src/main/kotlin/api/KotlinSignaturesLoading.kt Outdated
@martinbonnin martinbonnin force-pushed the const-false-positive branch from 7764ffb to ff78c5b Compare June 28, 2024 08:58
@fzhinkin fzhinkin self-requested a review June 28, 2024 09:30
Copy link
Copy Markdown
Collaborator

@fzhinkin fzhinkin left a comment

Choose a reason for hiding this comment

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

LGTM, thanks! I'll merge it later today

@fzhinkin fzhinkin changed the base branch from master to develop June 28, 2024 10:19
@fzhinkin fzhinkin merged commit 0dd6e41 into Kotlin:develop Jun 28, 2024
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.

2 participants