Skip to content

Conversation

@resyntax-ci
Copy link
Contributor

@resyntax-ci resyntax-ci bot commented May 7, 2025

Resyntax fixed 20 issues in 8 files.

  • Fixed 4 occurrences of let-to-define
  • Fixed 2 occurrences of if-let-to-cond
  • Fixed 2 occurrences of provide/contract-to-contract-out
  • Fixed 2 occurrences of if-begin-to-cond
  • Fixed 2 occurrences of map-to-for
  • Fixed 1 occurrence of always-throwing-cond-to-when
  • Fixed 1 occurrence of define-lambda-to-define
  • Fixed 1 occurrence of or-hash-ref-set!-to-hash-ref!
  • Fixed 1 occurrence of cond-let-to-cond-define
  • Fixed 1 occurrence of when-expression-in-for-loop-to-when-keyword
  • Fixed 1 occurrence of always-throwing-if-to-when
  • Fixed 1 occurrence of quasiquote-to-append
  • Fixed 1 occurrence of inline-unnecessary-begin

resyntax-ci bot added 13 commits May 7, 2025 00:17
The `define` form supports a shorthand for defining functions.
Internal definitions are recommended instead of `let` expressions, to reduce nesting.
This `map` operation can be replaced with a `for/list` loop.
Using `cond` instead of `if` here makes `begin` unnecessary
This expression can be replaced with a simpler, equivalent `hash-ref!` expression.
This `begin` form can be flattened into the surrounding definition context.
`cond` with internal definitions is preferred over `if` with `let`, to reduce nesting
This quasiquotation is equialent to calling `append`.
Using `when` and `unless` is simpler than a conditional with an always-throwing branch.
The `provide/contract` form is a legacy form made obsolete by `contract-out`.
Using `when` and `unless` is simpler than a conditional with an always-throwing branch.
Internal definitions are recommended instead of `let` expressions, to reduce nesting.
Use the `#:when` keyword instead of `when` to reduce loop body indentation.
@mflatt
Copy link
Member

mflatt commented Jun 26, 2025

Conflicts with other PRs tjat I plan to merge.

@mflatt mflatt closed this Jun 26, 2025
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.

2 participants