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

ktlint isn't suppressed by ConstPropertyName #2441

Closed
Omico opened this issue Dec 19, 2023 · 1 comment · Fixed by #2442
Closed

ktlint isn't suppressed by ConstPropertyName #2441

Omico opened this issue Dec 19, 2023 · 1 comment · Fixed by #2442
Milestone
1.1

Comments

@Omico
Copy link

Omico commented Dec 19, 2023

image
IDEA will mark PropertyName as a redundant suppression in this situation.
@paul-dingemans
Copy link
Collaborator

Tnx for reporting. You're just in time for the next release ;-)

@paul-dingemans paul-dingemans added this to the 1.1 milestone Dec 19, 2023
paul-dingemans added a commit that referenced this issue Dec 19, 2023
@paul-dingemans
Fix documentation
5a37fb0 
Closes #2441
@paul-dingemans paul-dingemans mentioned this issue Dec 19, 2023
Merged
8 tasks
paul-dingemans added a commit that referenced this issue Dec 19, 2023
@paul-dingemans
Suppress property-naming rule via @Suppress("ConstPropertyName") (#…
519ecbc 
…2442)

* Suppress property-naming rule via `@Suppress("ConstPropertyName")`

Suppression via `@Suppress("PropertyName")` was already supported before.

Closes #2441
@Goooler Goooler mentioned this issue Mar 14, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
1.1
Development

Successfully merging a pull request may close this issue.

Suppress property-naming rule via @Suppress("ConstPropertyName") pinterest/ktlint
2 participants
@paul-dingemans @Omico