Skip to content

Automated Resyntax fixes #1448

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

Open
wants to merge 22 commits into
base: master
Choose a base branch
from
Open

Automated Resyntax fixes #1448

wants to merge 22 commits into from

Conversation

resyntax-ci[bot]
Copy link
Contributor

@resyntax-ci resyntax-ci bot commented Apr 25, 2025

Resyntax fixed 42 issues in 20 files.

  • Fixed 12 occurrences of let-to-define
  • Fixed 5 occurrences of cond-let-to-cond-define
  • Fixed 4 occurrences of define-lambda-to-define
  • Fixed 3 occurrences of if-else-false-to-and
  • Fixed 2 occurrences of nested-if-to-cond
  • Fixed 2 occurrences of inverted-when
  • Fixed 2 occurrences of map-to-for
  • Fixed 1 occurrence of zero-comparison-to-positive?
  • Fixed 1 occurrence of define-simple-macro-to-define-syntax-parse-rule
  • Fixed 1 occurrence of sort-with-keyed-comparator-to-sort-by-key
  • Fixed 1 occurrence of begin0-let-to-define-begin0
  • Fixed 1 occurrence of case-lambda-with-single-case-to-lambda
  • Fixed 1 occurrence of quasiquote-to-list
  • Fixed 1 occurrence of for/fold-result-keyword
  • Fixed 1 occurrence of define-let-to-double-define
  • Fixed 1 occurrence of if-begin-to-cond
  • Fixed 1 occurrence of equal-null-list-to-null-predicate
  • Fixed 1 occurrence of always-throwing-if-to-when
  • Fixed 1 occurrence of syntax-disarm-migration

resyntax-ci bot added 22 commits April 25, 2025 00:54
The `define-simple-macro` form has been renamed to `define-syntax-parse-rule`.
The `null?` predicate can be used to test for the empty list.
Internal definitions are recommended instead of `let` expressions, to reduce nesting.
Only one of the `for/fold` expression's result values is used. Use the `#:result` keyword to return just that result.
The `define` form supports a shorthand for defining functions.
This `map` operation can be replaced with a `for/list` loop.
Internal definitions are recommended instead of `let` expressions, to reduce nesting.
This negated `when` expression can be replaced by an `unless` expression.
This `if` expression can be refactored to an equivalent expression using `and`.
The `syntax-disarm` function is a legacy function that does nothing.
Using `cond` instead of `if` here makes `begin` unnecessary
The `let` expression in this `begin0` form can be extracted into the surrounding definition context.
This `case-lambda` form only has one case. Use a regular lambda instead.
Using `when` and `unless` is simpler than a conditional with an always-throwing branch.
This quasiquotation is equialent to a simple `list` call.
This `sort` expression can be replaced with a simpler, equivalent expression.
This expression is equivalent to calling the `positive?` predicate.
This `if` expression can be refactored to an equivalent expression using `and`.
Internal definitions are recommended instead of `let` expressions, to reduce nesting.
Internal definitions are recommended instead of `let` expressions, to reduce nesting.
This `if`-`else` chain can be converted to a `cond` expression.
This `let` expression can be pulled up into a `define` expression.
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.

0 participants