Nicely formatted irb output & prompt #28
Conversation
This enables accessing history with the up cursor key even between irb sessions
I did this initially because the pre-existing code to create the ~/.zshrc file was wrapped in a condition checking for the existance of the file. However I don't see why this condition is necessary. `touch` will only update the accessed and modified timestamps on a file if it already exists, and in all cases we are modifying the files at this time.
|
Nice. Can you copy n paste the part of the script that we can run pls? (to set it up) |
|
@keithdhd shell script just for this part here: https://gist.github.com/CraigMorton/315d0f206e1414804e639cde731dba24 |
|
I'm a bit nervous about how much we're making the script do...and introducing more things that could go wrong or that makes it trickier for all instructors to understand what's going on. It's getting beyond installation and into quite detailed configuration. Is this something we could add as an additional resource? |
|
Those are both good points, and I agree that minimising what the script does, to maximise understanding and minimise complexity, is a good idea. I think we need to take that to a more general discussion on what we want this script to be. I don't think this PR is the most egregious example of either incomprehensible, or complex / brittle code in the script. If we're going to maintain this script based on minimising things that could go wrong, we could probably use the system version of ruby, macOS 10.14 comes with 2.3 now, that would cut out a lot of rbenv complexity which has caused failed installs in the past. Do we need oh-my-zsh? Do we even really need zsh? We could also consider ripping out the AppleScript for checking if GUI apps are already installed, because euw, AppleScript... - I'm not suggesting we do any of these, just saying this line of discussion will grow arms and legs pretty quickly. This PR just makes IRB easier to use than it is by default, splash of colour, less irrelevant spam in the prompt. It creates config files in |
This PR adds some configuration files for IRB to format the output of IRB to be more colourful and readable.
I've also configured the prompt to take up less space in the terminal.
All @codeclan/instructors should probably configure their own IRB to be similar before students laptop's start having this config. You can run this section of the script to get it as I have set it up, or feel free to configure it however you like in
~/.irbrcand~/.aprc(replacinggem_install_or_update- a helper function in the installation script, with simplygem install)Before:
After:
