Skip to content

Automated Resyntax fixes #487

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

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

Automated Resyntax fixes #487

wants to merge 8 commits into from

Conversation

resyntax-ci[bot]
Copy link
Contributor

@resyntax-ci resyntax-ci bot commented Feb 26, 2025

Resyntax fixed 20 issues in 8 files.

  • Fixed 10 occurrences of map-to-for
  • Fixed 4 occurrences of provide/contract-to-contract-out
  • Fixed 1 occurrence of if-let-to-cond
  • Fixed 1 occurrence of let-to-define
  • Fixed 1 occurrence of quasiquote-to-list
  • Fixed 1 occurrence of arrow-contract-with-rest-to-arrow-contract-with-ellipses
  • Fixed 1 occurrence of for/fold-result-keyword
  • Fixed 1 occurrence of always-throwing-if-to-when

`cond` with internal definitions is preferred over `if` with `let`, to reduce nesting
This `map` operation can be replaced with a `for/list` loop.
This quasiquotation is equialent to a simple `list` call.
The `provide/contract` form is a legacy form made obsolete by `contract-out`.
…ellipses`

This `->*` contract can be rewritten using `->` with ellipses.
Internal definitions are recommended instead of `let` expressions, to reduce nesting.
Using `when` and `unless` is simpler than a conditional with an always-throwing branch.
Only one of the `for/fold` expression's result values is used. Use the `#:result` keyword to return just that result.
# 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.

0 participants