Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

feat: add code action for the ParenthesesOnZeroArityDefs check #76

Merged

Conversation

sineed
Copy link
Contributor

@sineed sineed commented Jul 8, 2023

Screen.Recording.2023-07-08.at.18.18.02.mov

}} = Schematic.unify(opts(), Map.new(opts))

function_definition = Enum.at(text, start.line)
new_text = String.replace(function_definition, "()", "")
Copy link
Contributor

Choose a reason for hiding this comment

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

I think this will not produce the correct result for something like

def foo(), do: bar()

Copy link
Contributor Author

Choose a reason for hiding this comment

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

fixed

@mhanberg
Copy link
Contributor

I have that one comment that I think needs addressed, and I pushed up some superficial changes that weren't worth leaving comments for

@sineed
Copy link
Contributor Author

sineed commented Jul 13, 2023

no problem with changes, I just followed the way it was coded for the ModuleDocFalse.

@mhanberg mhanberg merged commit f846c9d into elixir-tools:main Jul 28, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants