Skip to content

(WIP) Use define-deprecated-alias for legacy aliases #1460

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

Draft
wants to merge 9 commits into
base: master
Choose a base branch
from

Conversation

jackfirth
Copy link
Contributor

This is a (very) work-in-progress pull request to use the new define-deprecated-alias form to define the various aliases listed in Legacy Forms. This will allow Resyntax to automatically migrate users of these forms to their recommended replacements.

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Resyntax analyzed 2 files in this pull request and has added suggestions.

jackfirth and others added 2 commits June 28, 2025 00:15
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Resyntax analyzed 2 files in this pull request and found no issues.

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Resyntax analyzed 2 files in this pull request and found no issues.

@jackfirth
Copy link
Contributor Author

@samth These test failures confound me. Any idea what's going on here?

--------------------
Typechecker tests > tc-expr tests > 652 (case-lambda: (((a : Number) (b : Number)) (+ a b)))
ERROR
location:   <pkgs>/typed-racket-test/unit-tests/typecheck-tests.rkt:652:8

typed-racket-test/unit-tests/typecheck-tests.rkt:652:35: type declaration: must be used in a definition context
  in: :
--------------------

@samth
Copy link
Member

samth commented Jun 29, 2025

That's some of the most fragile code in TR. Is that test using the old alias or the new name?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants