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

The define-lambda-to-define rule ought to be split #433

Open
jackfirth opened this issue Dec 11, 2024 · 0 comments
Open

The define-lambda-to-define rule ought to be split #433

jackfirth opened this issue Dec 11, 2024 · 0 comments
Labels
existing lint Issues or pull requests relating to existing lints

Comments

@jackfirth
Copy link
Owner

The define-lambda-to-define rule works on both ordinary (define id (lambda ...)) forms and also (define (func ...) (lambda ...)) forms. But the rule description is written assuming it's for the (define id (lambda ...)) form. Covering these two cases with separate rules and separate messages would make this refactoring a little more readable.

@jackfirth jackfirth added the new lint Issues suggesting new lints or pull requests implementing new lints label Dec 11, 2024
@jackfirth jackfirth added existing lint Issues or pull requests relating to existing lints and removed new lint Issues suggesting new lints or pull requests implementing new lints labels Dec 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
existing lint Issues or pull requests relating to existing lints
Projects
None yet
Development

No branches or pull requests

1 participant