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

Deleting NoItParamInMultilineLambdaRule #510

Merged
merged 1 commit into from
Jul 5, 2019

Conversation

shashachu
Copy link
Contributor

It was based on a styleguide discussion (Kotlin/kotlin-style-guide#10) that never made it into the official styleguide.

Copy link
Collaborator

@Tapchicoma Tapchicoma left a comment

Choose a reason for hiding this comment

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

LGTM 👍

It was based on a styleguide discussion (Kotlin/kotlin-style-guide#10) that never made it into the official styleguide.
@shashachu shashachu merged commit 9b8ead8 into pinterest:master Jul 5, 2019
sowmyav24 pushed a commit to sowmyav24/ktlint that referenced this pull request Jul 18, 2019
It was based on a styleguide discussion (Kotlin/kotlin-style-guide#10) that never made it into the official styleguide.
@shashachu shashachu deleted the delete-no-it-rule branch August 2, 2019 22:16
@Zhuinden
Copy link

Zhuinden commented Feb 7, 2020

Well that's kind of disappointing.

I'll just leave this here: https://twitter.com/kotlin/status/1220383288109535232

@Tapchicoma
Copy link
Collaborator

@Zhuinden you could add it as a custom rule. If you think that it should be in standard ruleset, please open a new issue.

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.

None yet

3 participants