-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Add 'compile' command as an alias of the 'build' command. #2725
Conversation
@@ -8,6 +8,7 @@ class Crystal::Command | |||
Command: | |||
init generate a new project | |||
build compile program |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actually perhaps we should even already remove it here.
options.shift | ||
build | ||
compile command |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think the usage saying "compile" when "build" is still used would actually be a good thing :)
Great, now some squashing would be great here too ;) |
Remove 'build' command from compiler usage. Unify build/compile handling. Update compiler spec to use new 'compile' command. Compile/build command's name now always 'compile'.
Squashed! |
I thought we didn't like aliases? |
We don't, but then this is so essential that we need to phase it out IMO. So the current plan is to add this as an alias, update all references we can update to compile and then at some later release remove build. |
Also our own |
Thanks! There was no indication about a deprecation of the build command. |
Oh, a deprecation warning is very good indeed, so users can learn about the new command. |
Travis is green! |
Add 'compile' command as an alias of the 'build' command.
Thank you! |
Woot! |
Fixes #502