-
-
Notifications
You must be signed in to change notification settings - Fork 93
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
Refactor blueboxes name to signature-box #286
base: master
Are you sure you want to change the base?
Conversation
Simple find-and-replace to change the name for blueboxes to signature-box
While I agree with the renaming in general, please keep Similarly, I'm not sure if renaming the Scribble file name is a good idea. Someone might link to https://docs.racket-lang.org/scribble/blueboxes.html literally, and this change would break the link. |
I agree with @sorawee -- we probably don't want to change any of the existing files or exported names. I think it makes sense to change the documentation to call them "signature boxes" and it may make sense (but is less obviously good) to add new files that just reexport the existing functions under new names (or to rename the existing files and then add backwards compatibility files). As for the error message -- I think you probably want to work with a more recent version of Racket before trying to investigate that too deeply. Assuming you have build-essentials installed (and maybe a few other packages) you should be able to simply do |
This tutorial by Ben may help: Tutorial: Contributing to Racket |
Not to talk past the close, but, changing the required module name and function names would break more things than just Racket Mode.
I totally agree that "bluebox" has been kind of an obscure term ever since the docs got a new style sheet, years ago. 😄 So I definitely think there's merit to changing the docs to help. Like maybe define "bluebox" as a |
Dang I can't recall the Anyway, here's one list: https://github.com/search?p=2&q=blueboxes-cache&type=Code That list isn't perfect. It's too long because it includes many forked repos (although some of those forks might need to be changed, eventually). It's too short because it's only code on GitHub. But generally speaking Racket "never" removes modules or files, and breaks existing programs. It might add new, preferred alternatives, or aliases. |
Thanks @greghendershott! I had no idea this code could have such far reaching impact. 😄 I appreciate all the pointers on this from everyone. |
Simple find-and-replace to change the name for blueboxes to signature-box.
(Quick note: this is my first time messing with racket, so apologies if anything is incorrect/wonky/etc.)
Followed the steps outlined #206 and then ran the tests using
raco test scribble-test
and saw them all pass with racket version v7.9.0.22 [cs] on Ubuntu 20.04