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

New rule AvoidMisleadingRecursiveKeywordInNonRecursiveFuncs #650

Closed
knocte opened this issue Dec 25, 2023 · 0 comments · Fixed by #652
Closed

New rule AvoidMisleadingRecursiveKeywordInNonRecursiveFuncs #650

knocte opened this issue Dec 25, 2023 · 0 comments · Fixed by #652

Comments

@knocte
Copy link
Collaborator

knocte commented Dec 25, 2023 •
edited
Loading

Testcases:

Should error:

let rec Foo someParam =
    ()

Should not error:

let rec Foo someParam =
    if someParam then
        Foo()
    else
        ()
@knocte knocte changed the title AvoidMisleadingRecursiveKeywordInNonRecursiveFuncs New rule AvoidMisleadingRecursiveKeywordInNonRecursiveFuncs Dec 25, 2023
@knocte knocte mentioned this issue Dec 25, 2023
Closed
@Mersho Mersho mentioned this issue Dec 26, 2023
Merged
knocte added a commit that referenced this issue Jan 8, 2024
@knocte
Merge PR #652 from Mersho/fix-650
aeadd7f 
New rule UnneededRecKeyword.

Fixes #650
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
No milestone
Development

Successfully merging a pull request may close this issue.

New rule UnneededRecKeyword Mersho/FSharpLint
1 participant
@knocte