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

Automated Resyntax fixes #705

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

Automated Resyntax fixes #705

wants to merge 8 commits into from

Conversation

resyntax-ci[bot]
Copy link
Contributor

@resyntax-ci resyntax-ci bot commented Dec 29, 2024

Resyntax fixed 20 issues in 9 files.

  • Fixed 8 occurrences of tidy-require
  • Fixed 6 occurrences of let-to-define
  • Fixed 1 occurrence of read-line-any
  • Fixed 1 occurrence of define-lambda-to-define
  • Fixed 1 occurrence of zero-comparison-to-positive?
  • Fixed 1 occurrence of if-let-to-cond
  • Fixed 1 occurrence of cond-else-cond-to-cond
  • Fixed 1 occurrence of quasiquote-to-list

Keep imports in `require` sorted and grouped by phase, with collections before files.
The `define` form supports a shorthand for defining functions.
Internal definitions are recommended instead of `let` expressions, to reduce nesting.
`cond` with internal definitions is preferred over `if` with `let`, to reduce nesting
Specify a line mode of `'any` with `read-line` to avoid differences between Windows and other platforms.
This quasiquotation is equialent to a simple `list` call.
The `else` clause of this `cond` expression is another `cond` expression and can be flattened.
This expression is equivalent to calling the `positive?` predicate.
@spdegabrielle
Copy link
Member

@jackfirth are these abandoned? I think there have been more recent ones that have been merged?

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.

1 participant