Skip to content

Renaming required identifier changes require path #491

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
LLazarek opened this issue Jun 15, 2021 · 2 comments
Open

Renaming required identifier changes require path #491

LLazarek opened this issue Jun 15, 2021 · 2 comments
Labels
bug check-syntax issues related to check-syntax, background expansion, drawing nifty arrows, etc.

Comments

@LLazarek
Copy link

Renaming a required identifier changes the module path in that file's require.

Example: two files, a.rkt and b.rkt

;; a.rkt
#lang racket
(provide a)
(define a 5)

;; b.rkt
#lang racket
(require "a.rkt")
(+ a 2)

Opening b.rkt, right clicking on a on line 3, and renaming it to foo changes the line 2 of b.rkt to (require foo).

What I expected: it would leave the require path alone.

Racket version: 8.1.0.7 [cs]

@sorawee
Copy link
Contributor

sorawee commented Jun 16, 2021

Relevant discussion: #417

@jackfirth jackfirth added bug check-syntax issues related to check-syntax, background expansion, drawing nifty arrows, etc. labels Jun 24, 2021
@rfindler
Copy link
Member

This no longer offers a renaming option. Probably it is okay to close this, given #417.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug check-syntax issues related to check-syntax, background expansion, drawing nifty arrows, etc.
Projects
None yet
Development

No branches or pull requests

4 participants